[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