[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