diff options
| author | David Spickett <david.spickett@linaro.org> | 2025-06-12 13:10:04 +0000 |
|---|---|---|
| committer | David Spickett <david.spickett@linaro.org> | 2025-06-12 13:17:24 +0000 |
| commit | d49bc5e621c8931679b232fa28abfc89a171105e (patch) | |
| tree | 2393abcb7ad0cd02b5a4592faf832b6215592c15 /lldb/source/Commands/CommandObjectMemory.cpp | |
| parent | ce747a16328b2fbc365e1cb1cb01cb400c2c1b4c (diff) | |
| download | llvm-d49bc5e621c8931679b232fa28abfc89a171105e.zip llvm-d49bc5e621c8931679b232fa28abfc89a171105e.tar.gz llvm-d49bc5e621c8931679b232fa28abfc89a171105e.tar.bz2 | |
[llvm][MemProf] Correct position of LLVM_ABI macro in computeFrameHistogram
The previous placement resulted in this warning when using g++-13:
/home/david.spickett/llvm-project/llvm/include/llvm/Support/Compiler.h:120:43: warning: attribute ignored [-Wattributes]
120 | #define LLVM_ATTRIBUTE_VISIBILITY_DEFAULT [[gnu::visibility("default")]]
| ^
/home/david.spickett/llvm-project/llvm/include/llvm/Support/Compiler.h:213:18: note: in expansion of macro ‘LLVM_ATTRIBUTE_VISIBILITY_DEFAULT’
213 | #define LLVM_ABI LLVM_ATTRIBUTE_VISIBILITY_DEFAULT
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/david.spickett/llvm-project/llvm/lib/ProfileData/MemProfRadixTree.cpp:245:5: note: in expansion of macro ‘LLVM_ABI’
245 | LLVM_ABI computeFrameHistogram<FrameId>(
| ^~~~~~~~
/home/david.spickett/llvm-project/llvm/include/llvm/Support/Compiler.h:120:43: note: an attribute that appertains to a type-specifier is ignored
120 | #define LLVM_ATTRIBUTE_VISIBILITY_DEFAULT [[gnu::visibility("default")]]
| ^
According to the interface guide, that macro should go before the return
type to be effective.
https://llvm.org/docs/InterfaceExportAnnotations.html#specialized-template-functions
Diffstat (limited to 'lldb/source/Commands/CommandObjectMemory.cpp')
0 files changed, 0 insertions, 0 deletions