llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-driver Author: Prabhu Rajasekaran (Prabhuk) <details> <summary>Changes</summary> Set MS ABI as default ABI for UEFI. --- Full diff: https://github.com/llvm/llvm-project/pull/138364.diff 1 Files Affected: - (modified) clang/lib/Driver/ToolChains/Arch/X86.cpp (+1-1) ``````````diff diff --git a/clang/lib/Driver/ToolChains/Arch/X86.cpp b/clang/lib/Driver/ToolChains/Arch/X86.cpp index e6ac3a9e4b350..eea0c9e2ae6e0 100644 --- a/clang/lib/Driver/ToolChains/Arch/X86.cpp +++ b/clang/lib/Driver/ToolChains/Arch/X86.cpp @@ -122,7 +122,7 @@ void x86::getX86TargetFeatures(const Driver &D, const llvm::Triple &Triple, // Claim and report unsupported -mabi=. Note: we don't support "sysv_abi" or // "ms_abi" as default function attributes. if (const Arg *A = Args.getLastArg(clang::driver::options::OPT_mabi_EQ)) { - StringRef DefaultAbi = Triple.isOSWindows() ? "ms" : "sysv"; + StringRef DefaultAbi = Triple.isOSWindowsOrUEFI() ? "ms" : "sysv"; if (A->getValue() != DefaultAbi) D.Diag(diag::err_drv_unsupported_opt_for_target) << A->getSpelling() << Triple.getTriple(); `````````` </details> https://github.com/llvm/llvm-project/pull/138364 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
