[clang] [clang][NFC] Update isAuxBuiltinID comment (PR #128038)
via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 20 09:58:53 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Nick Sarnie (sarnex)
<details>
<summary>Changes</summary>
Clarify behavior of the function when the builtin is also supported on the main target.
Based on feedback from https://github.com/llvm/llvm-project/pull/126324
---
Full diff: https://github.com/llvm/llvm-project/pull/128038.diff
1 Files Affected:
- (modified) clang/include/clang/Basic/Builtins.h (+3-1)
``````````diff
diff --git a/clang/include/clang/Basic/Builtins.h b/clang/include/clang/Basic/Builtins.h
index 6d29b4315e5a7..91a03c3a51da2 100644
--- a/clang/include/clang/Basic/Builtins.h
+++ b/clang/include/clang/Basic/Builtins.h
@@ -408,7 +408,9 @@ class Context {
unsigned getRequiredVectorWidth(unsigned ID) const;
- /// Return true if builtin ID belongs to AuxTarget.
+ /// Return true if builtin ID belongs to only the AuxTarget.
+ /// Return false if builtin ID belongs to both the primary
+ /// and auxiliary target.
bool isAuxBuiltinID(unsigned ID) const {
return ID >= (Builtin::FirstTSBuiltin + NumTargetBuiltins);
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/128038
More information about the cfe-commits
mailing list