[clang] [llvm] Rename wasm32-wasi to wasm32-wasip1. (PR #165345)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 27 19:21:33 PDT 2025
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 origin/main HEAD --extensions cpp,h -- clang/lib/Basic/Targets.cpp clang/lib/Basic/Targets/OSTargets.h clang/lib/Driver/ToolChains/WebAssembly.cpp clang/lib/Lex/InitHeaderSearch.cpp llvm/include/llvm/TargetParser/Triple.h llvm/lib/TargetParser/Triple.cpp llvm/unittests/TargetParser/TripleTest.cpp --diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/lib/Basic/Targets.cpp b/clang/lib/Basic/Targets.cpp
index f27f53fc1..35c6e278d 100644
--- a/clang/lib/Basic/Targets.cpp
+++ b/clang/lib/Basic/Targets.cpp
@@ -702,17 +702,17 @@ std::unique_ptr<TargetInfo> AllocateTarget(const llvm::Triple &Triple,
!Triple.isOSBinFormatWasm())
return nullptr;
switch (os) {
- case llvm::Triple::WASI: // Treat "wasi" as "wasip1" for now.
- case llvm::Triple::WASIp1:
+ case llvm::Triple::WASI: // Treat "wasi" as "wasip1" for now.
+ case llvm::Triple::WASIp1:
return std::make_unique<WASIP1TargetInfo<WebAssembly32TargetInfo>>(Triple,
Opts);
- case llvm::Triple::WASIp2:
+ case llvm::Triple::WASIp2:
return std::make_unique<WASIP2TargetInfo<WebAssembly32TargetInfo>>(Triple,
Opts);
- case llvm::Triple::WASIp3:
+ case llvm::Triple::WASIp3:
return std::make_unique<WASIP3TargetInfo<WebAssembly32TargetInfo>>(Triple,
Opts);
- case llvm::Triple::Emscripten:
+ case llvm::Triple::Emscripten:
return std::make_unique<EmscriptenTargetInfo<WebAssembly32TargetInfo>>(
Triple, Opts);
@@ -731,17 +731,17 @@ std::unique_ptr<TargetInfo> AllocateTarget(const llvm::Triple &Triple,
!Triple.isOSBinFormatWasm())
return nullptr;
switch (os) {
- case llvm::Triple::WASI: // Treat "wasi" as "wasip1" for now.
- case llvm::Triple::WASIp1:
+ case llvm::Triple::WASI: // Treat "wasi" as "wasip1" for now.
+ case llvm::Triple::WASIp1:
return std::make_unique<WASIP1TargetInfo<WebAssembly64TargetInfo>>(Triple,
Opts);
- case llvm::Triple::WASIp2:
+ case llvm::Triple::WASIp2:
return std::make_unique<WASIP2TargetInfo<WebAssembly64TargetInfo>>(Triple,
Opts);
- case llvm::Triple::WASIp3:
+ case llvm::Triple::WASIp3:
return std::make_unique<WASIP3TargetInfo<WebAssembly64TargetInfo>>(Triple,
Opts);
- case llvm::Triple::Emscripten:
+ case llvm::Triple::Emscripten:
return std::make_unique<EmscriptenTargetInfo<WebAssembly64TargetInfo>>(
Triple, Opts);
case llvm::Triple::UnknownOS:
diff --git a/llvm/include/llvm/TargetParser/Triple.h b/llvm/include/llvm/TargetParser/Triple.h
index 89bf990d8..81d287989 100644
--- a/llvm/include/llvm/TargetParser/Triple.h
+++ b/llvm/include/llvm/TargetParser/Triple.h
@@ -747,7 +747,7 @@ public:
/// Tests whether the OS is WASI.
bool isOSWASI() const {
return getOS() == Triple::WASI || getOS() == Triple::WASIp1 ||
- getOS() == Triple::WASIp2 || getOS() == Triple::WASIp3;
+ getOS() == Triple::WASIp2 || getOS() == Triple::WASIp3;
}
/// Tests whether the OS is Emscripten.
diff --git a/llvm/lib/TargetParser/Triple.cpp b/llvm/lib/TargetParser/Triple.cpp
index 80d500dcc..6bde5935f 100644
--- a/llvm/lib/TargetParser/Triple.cpp
+++ b/llvm/lib/TargetParser/Triple.cpp
@@ -324,9 +324,12 @@ StringRef Triple::getOSTypeName(OSType Kind) {
case TvOS: return "tvos";
case UEFI: return "uefi";
case WASI: return "wasi";
- case WASIp1: return "wasip1";
- case WASIp2: return "wasip2";
- case WASIp3: return "wasip3";
+ case WASIp1:
+ return "wasip1";
+ case WASIp2:
+ return "wasip2";
+ case WASIp3:
+ return "wasip3";
case WatchOS: return "watchos";
case Win32: return "windows";
case ZOS: return "zos";
``````````
</details>
https://github.com/llvm/llvm-project/pull/165345
More information about the llvm-commits
mailing list