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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to