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
Build idt from compnerd/ids, then for each changed header:
idt -p build/ --main-file <matching-source.cpp> \
--apply-fixits --inplace <header>
``````````
</details>
<details>
<summary>
View the diff from ids-check here.
</summary>
``````````diff
diff --git a/llvm/include/llvm/IR/Module.h b/llvm/include/llvm/IR/Module.h
index a82ac363d..63d101389 100644
--- a/llvm/include/llvm/IR/Module.h
+++ b/llvm/include/llvm/IR/Module.h
@@ -175,10 +175,10 @@ public:
/// Set a property using a string name.
/// Returns whether the property name was valid.
- bool set(StringRef Name, std::string Value);
+ LLVM_ABI bool set(StringRef Name, std::string Value);
/// Get a list of set properties as pairs of key and value.
- SmallVector<std::pair<StringRef, StringRef>> getAsStrings() const;
+ LLVM_ABI SmallVector<std::pair<StringRef, StringRef>> getAsStrings() const;
bool operator==(const GlobalAsmProperties &Other) const {
return TargetFeatures == Other.TargetFeatures &&
``````````
</details>
https://github.com/llvm/llvm-project/pull/204548
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
