[clang] [clang] UEFI default ABI (PR #138364)
via cfe-commits
cfe-commits at lists.llvm.org
Fri May 2 16:40:35 PDT 2025
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
More information about the cfe-commits
mailing list