diff options
| author | Jason Molenda <jmolenda@apple.com> | 2024-05-02 15:52:24 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-05-02 15:52:24 -0700 |
| commit | 2d15855adf6afac3f4171a24fd18b65a32eedf78 (patch) | |
| tree | 5349f9ee762433a72f8ce2931b2c267eac303b38 /llvm/lib/Bitcode/Reader/BitcodeReader.cpp | |
| parent | 9dca7dde3ff2e1471ef4fc145b56d7bea4d06ae8 (diff) | |
[lldb] [debugserver] address preprocessor warning, extra arg (#90808)
In DNBArchImplARM64.cpp I'm doing
```
#if __has_feature(ptrauth_calls) && defined(__LP64__)
```
And the preprocessor warns that this is not defined behavior. This
checks if ptrauth_calls is available and if this is being compiled
64-bit (i.e. arm64e), and defines a single DEBUGSERVER_IS_ARM64E when
those are both true.
I did have to duplicate one DNBLogThreaded() call which itself is a
macro, and using an ifdef in the middle of macro arguments also got me a
warning from the preprocessor.
While testing this for all the different targets, I found a DNBError
initialization that accepts a c-string but I'm passing in a printf-style
formatter c-string and an argument. Create the string before the call
and pass in the constructed string.
rdar://127129242
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp')
0 files changed, 0 insertions, 0 deletions
