[clang] cc45365 - [clang][NFC] Update isAuxBuiltinID comment (#128038)

via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 20 12:30:59 PST 2025


Author: Nick Sarnie
Date: 2025-02-20T20:30:54Z
New Revision: cc45365ab9065bc9e8ea48744f761b7ac30a5634

URL: https://github.com/llvm/llvm-project/commit/cc45365ab9065bc9e8ea48744f761b7ac30a5634
DIFF: https://github.com/llvm/llvm-project/commit/cc45365ab9065bc9e8ea48744f761b7ac30a5634.diff

LOG: [clang][NFC] Update isAuxBuiltinID comment (#128038)

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

---------

Signed-off-by: Sarnie, Nick <nick.sarnie at intel.com>

Added: 
    

Modified: 
    clang/include/clang/Basic/Builtins.h

Removed: 
    


################################################################################
diff  --git a/clang/include/clang/Basic/Builtins.h b/clang/include/clang/Basic/Builtins.h
index 6d29b4315e5a7..d1c1ab44417fa 100644
--- a/clang/include/clang/Basic/Builtins.h
+++ b/clang/include/clang/Basic/Builtins.h
@@ -408,7 +408,8 @@ class Context {
 
   unsigned getRequiredVectorWidth(unsigned ID) const;
 
-  /// Return true if builtin ID belongs to AuxTarget.
+  /// Return true if the builtin ID belongs exclusively to the AuxTarget,
+  /// and false if it belongs to both primary and aux target, or neither.
   bool isAuxBuiltinID(unsigned ID) const {
     return ID >= (Builtin::FirstTSBuiltin + NumTargetBuiltins);
   }


        


More information about the cfe-commits mailing list