[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