Revert "[Clang] Reland '__has_builtin should return false for aux triple builtins' (llvm#126324)"

This reverts commit 0efcb83.

Author: ranapratap55

clang/docs/LanguageExtensions.rst

@@ -68,9 +68,6 @@ It can be used like this:
   ``__has_builtin`` should not be used to detect support for a builtin macro;
   use ``#ifdef`` instead.
 
-  When compiling with target offloading, ``__has_builtin`` only considers the
-  currently active target.
-
 ``__has_constexpr_builtin``
 ---------------------------
 

clang/docs/ReleaseNotes.rst

@@ -37,8 +37,6 @@ Potentially Breaking Changes
 C/C++ Language Potentially Breaking Changes
 -------------------------------------------
 
-- The ``__has_builtin`` function now only considers the currently active target when being used with target offloading.
-
 C++ Specific Potentially Breaking Changes
 -----------------------------------------
 - For C++20 modules, the Reduced BMI mode will be the default option. This may introduce

clang/lib/Lex/PPMacroExpansion.cpp

@@ -1760,8 +1760,7 @@ void Preprocessor::ExpandBuiltinMacro(Token &Tok) {
               Tok, *this, diag::err_feature_check_malformed);
           if (!II)
             return false;
-          unsigned BuiltinID = II->getBuiltinID();
-          if (BuiltinID != 0) {
+          else if (II->getBuiltinID() != 0) {
             switch (II->getBuiltinID()) {
             case Builtin::BI__builtin_cpu_is:
               return getTargetInfo().supportsCpuIs();
@@ -1775,11 +1774,8 @@ void Preprocessor::ExpandBuiltinMacro(Token &Tok) {
               // usual allocation and deallocation functions. Required by libc++
               return 201802;
             default:
-              // __has_builtin should return false for aux builtins.
-              if (getBuiltinInfo().isAuxBuiltinID(BuiltinID))
-                return false;
               return Builtin::evaluateRequiredTargetFeatures(
-                  getBuiltinInfo().getRequiredFeatures(BuiltinID),
+                  getBuiltinInfo().getRequiredFeatures(II->getBuiltinID()),
                   getTargetInfo().getTargetOpts().FeatureMap);
             }
             return true;

clang/test/Headers/__cpuidex_conflict.c

@@ -3,10 +3,7 @@
 // RUN: %clang_cc1 %s -ffreestanding -fms-extensions -fms-compatibility \
 // RUN:  -fms-compatibility-version=19.00 -triple x86_64-pc-windows-msvc -emit-llvm -o -
 // %clang_cc1 %s -ffreestanding -triple x86_64-w64-windows-gnu -fms-extensions -emit-llvm -o -
-//
-// FIXME: See https://github.com/llvm/llvm-project/pull/121839 and
-// FIXME: https://github.com/llvm/llvm-project/pull/126324
-// RUN: not %clang_cc1 %s -ffreestanding -fopenmp -fopenmp-is-target-device -aux-triple x86_64-unknown-linux-gnu
+// RUN: %clang_cc1 %s -ffreestanding -fopenmp -fopenmp-is-target-device -aux-triple x86_64-unknown-linux-gnu
 
 typedef __SIZE_TYPE__ size_t;