github-actions[bot] wrote:
<!--LLVM IDS CHECK COMMENT-->
:warning: LLVM ABI annotation checker, ids-check found issues in your code.
:warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git diff origin/main HEAD -- 'llvm/include/llvm/**/*.h'
'llvm/include/llvm-c/**/*.h' 'llvm/include/llvm/Demangle/**/*.h'
Then run idt on the changed files with appropriate --export-macro and
--include-header flags.
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from ids-check here.
</summary>
``````````diff
diff --git a/llvm/include/llvm/CodeGen/TargetLoweringObjectFileImpl.h
b/llvm/include/llvm/CodeGen/TargetLoweringObjectFileImpl.h
index fb0422820..39588cad6 100644
--- a/llvm/include/llvm/CodeGen/TargetLoweringObjectFileImpl.h
+++ b/llvm/include/llvm/CodeGen/TargetLoweringObjectFileImpl.h
@@ -14,6 +14,7 @@
#ifndef LLVM_CODEGEN_TARGETLOWERINGOBJECTFILEIMPL_H
#define LLVM_CODEGEN_TARGETLOWERINGOBJECTFILEIMPL_H
+#include "llvm/Support/Compiler.h"
#include "llvm/ADT/SmallPtrSet.h"
#include "llvm/BinaryFormat/XCOFF.h"
#include "llvm/MC/MCExpr.h"
@@ -31,7 +32,7 @@ class MCSymbol;
class Module;
class TargetMachine;
-class TargetLoweringObjectFileELF : public TargetLoweringObjectFile {
+class LLVM_ABI TargetLoweringObjectFileELF : public TargetLoweringObjectFile {
bool UseInitArray = false;
mutable unsigned NextUniqueID = 1; // ID 0 is reserved for execute-only
sections
SmallPtrSet<GlobalObject *, 2> Used;
@@ -130,7 +131,7 @@ public:
MCSection *getSectionForCommandLines() const override;
};
-class TargetLoweringObjectFileMachO : public TargetLoweringObjectFile {
+class LLVM_ABI TargetLoweringObjectFileMachO : public TargetLoweringObjectFile
{
public:
TargetLoweringObjectFileMachO();
~TargetLoweringObjectFileMachO() override = default;
@@ -180,7 +181,7 @@ public:
MCSection *getSectionForCommandLines() const override;
};
-class TargetLoweringObjectFileCOFF : public TargetLoweringObjectFile {
+class LLVM_ABI TargetLoweringObjectFileCOFF : public TargetLoweringObjectFile {
mutable unsigned NextUniqueID = 0;
const TargetMachine *TM = nullptr;
@@ -225,7 +226,7 @@ public:
Align &Alignment) const override;
};
-class TargetLoweringObjectFileWasm : public TargetLoweringObjectFile {
+class LLVM_ABI TargetLoweringObjectFileWasm : public TargetLoweringObjectFile {
mutable unsigned NextUniqueID = 0;
SmallPtrSet<GlobalObject *, 2> Used;
@@ -251,7 +252,7 @@ public:
const MCSymbol *KeySym) const override;
};
-class TargetLoweringObjectFileXCOFF : public TargetLoweringObjectFile {
+class LLVM_ABI TargetLoweringObjectFileXCOFF : public TargetLoweringObjectFile
{
public:
TargetLoweringObjectFileXCOFF() = default;
~TargetLoweringObjectFileXCOFF() override = default;
@@ -315,7 +316,7 @@ public:
const TargetMachine &TM) const override;
};
-class TargetLoweringObjectFileGOFF : public TargetLoweringObjectFile {
+class LLVM_ABI TargetLoweringObjectFileGOFF : public TargetLoweringObjectFile {
std::string DefaultRootSDName;
std::string DefaultADAPRName;
``````````
</details>
https://github.com/llvm/llvm-project/pull/153049
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
