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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
