[clang] [clang][WebAssembly] Support the `-m(no-)red-zone` flag. (PR #119997)

via cfe-commits cfe-commits at lists.llvm.org
Sat Dec 14 21:19:30 PST 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang-driver

Author: Alex Rønne Petersen (alexrp)

<details>
<summary>Changes</summary>

LLVM's WebAssembly backend supports the `noredzone` function attribute, but this support was not exposed in Clang.

Note: I don't have commit access.

---
Full diff: https://github.com/llvm/llvm-project/pull/119997.diff


1 Files Affected:

- (modified) clang/lib/Driver/ToolChains/Clang.cpp (+3) 


``````````diff
diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp
index 7ef55a33547c50..21117cd71b651e 100644
--- a/clang/lib/Driver/ToolChains/Clang.cpp
+++ b/clang/lib/Driver/ToolChains/Clang.cpp
@@ -2404,6 +2404,9 @@ void Clang::AddWebAssemblyTargetArgs(const ArgList &Args,
   if (!Args.hasArg(options::OPT_fvisibility_EQ,
                    options::OPT_fvisibility_ms_compat))
     CmdArgs.push_back("-fvisibility=hidden");
+
+  if (!Args.hasFlag(options::OPT_mred_zone, options::OPT_mno_red_zone, true))
+    CmdArgs.push_back("-disable-red-zone");
 }
 
 void Clang::AddVETargetArgs(const ArgList &Args, ArgStringList &CmdArgs) const {

``````````

</details>


https://github.com/llvm/llvm-project/pull/119997


More information about the cfe-commits mailing list