[clang] [llvm] [WebAssembly] Add `-i128:128` to the `datalayout` string. (PR #119204)

via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 10 07:58:08 PST 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 e665e781dc3f8fac0400f6ac8e168805b0cdf74a dd2939affd576539d83dfa877ea30192b41353b0 --extensions h,c,cpp -- clang/lib/Basic/Targets/WebAssembly.h clang/test/CodeGen/target-data.c llvm/lib/IR/AutoUpgrade.cpp llvm/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp llvm/unittests/Bitcode/DataLayoutUpgradeTest.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/llvm/lib/IR/AutoUpgrade.cpp b/llvm/lib/IR/AutoUpgrade.cpp
index e98a4647a4..06e62bf7f9 100644
--- a/llvm/lib/IR/AutoUpgrade.cpp
+++ b/llvm/lib/IR/AutoUpgrade.cpp
@@ -5559,7 +5559,8 @@ std::string llvm::UpgradeDataLayoutString(StringRef DL, StringRef TT) {
     return Res;
   }
 
-  if (T.isSPARC() || (T.isMIPS64() && !DL.contains("m:m")) || T.isPPC64() || T.isWasm()) {
+  if (T.isSPARC() || (T.isMIPS64() && !DL.contains("m:m")) || T.isPPC64() ||
+      T.isWasm()) {
     // Mips64 with o32 ABI did not add "-i128:128".
     // Add "-i128:128"
     std::string I64 = "-i64:64";

``````````

</details>


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


More information about the llvm-commits mailing list