[clang] [WebAssembly] Disable running `wasm-opt` on components (PR #98373)
via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 10 12:59:08 PDT 2024
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff 6e86e11148474e4ecd49dbf0ca5dd9caddcdbd11 177fe1d4df4e9f02382c2f8f5dce9de4d935d763 --extensions cpp -- clang/lib/Driver/ToolChains/WebAssembly.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/lib/Driver/ToolChains/WebAssembly.cpp b/clang/lib/Driver/ToolChains/WebAssembly.cpp
index 2753645f47..2ad2513dbc 100644
--- a/clang/lib/Driver/ToolChains/WebAssembly.cpp
+++ b/clang/lib/Driver/ToolChains/WebAssembly.cpp
@@ -166,7 +166,8 @@ void wasm::Linker::ConstructJob(Compilation &C, const JobAction &JA,
// components at this time. Retain the historical default otherwise, though,
// of running `wasm-opt` by default.
bool WasmOptDefault = !IsWasip2(ToolChain.getTriple());
- bool RunWasmOpt = Args.hasFlag(options::OPT_wasm_opt, options::OPT_no_wasm_opt, WasmOptDefault);
+ bool RunWasmOpt = Args.hasFlag(options::OPT_wasm_opt,
+ options::OPT_no_wasm_opt, WasmOptDefault);
// If wasm-opt is enabled and optimizations are happening look for the
// `wasm-opt` program. If it's not found auto-disable it.
``````````
</details>
https://github.com/llvm/llvm-project/pull/98373
More information about the cfe-commits
mailing list