[llvm] [ABI] Add DefaultTargetInfo for default argument/return classification (PR #225009)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 23 03:24:17 PDT 2026
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/ABI/DefaultTargetInfo.h b/llvm/include/llvm/ABI/DefaultTargetInfo.h
index 1df4191ac..0be2aea77 100644
--- a/llvm/include/llvm/ABI/DefaultTargetInfo.h
+++ b/llvm/include/llvm/ABI/DefaultTargetInfo.h
@@ -22,14 +22,14 @@ namespace llvm {
namespace abi {
/// Self-consistent classification that conforms to no particular ABI.
-class DefaultTargetInfo : public TargetInfo {
+class LLVM_ABI DefaultTargetInfo : public TargetInfo {
public:
using TargetInfo::TargetInfo;
- LLVM_ABI ArgInfo classifyArgumentType(const Type *Ty) const;
- LLVM_ABI ArgInfo classifyReturnType(const Type *RetTy) const;
+ ArgInfo classifyArgumentType(const Type *Ty) const;
+ ArgInfo classifyReturnType(const Type *RetTy) const;
- LLVM_ABI void computeInfo(FunctionInfo &FI) const override;
+ void computeInfo(FunctionInfo &FI) const override;
};
} // namespace abi
``````````
</details>
https://github.com/llvm/llvm-project/pull/225009
More information about the llvm-commits
mailing list