[Openmp-commits] [openmp] [OpenMP][Build][Wasm][116552] Fixed build problem when compiling with Emscripten on Windows (PR #116874)

via Openmp-commits openmp-commits at lists.llvm.org
Tue Nov 19 12:52:40 PST 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {darker}-->


:warning: Python code formatter, darker found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
darker --check --diff -r def22f4e718daa74c2d0c03a32e32d4913a46278...2a624fd108db2ceaf3f08afc3a7f551e27125b23 openmp/runtime/tools/message-converter.py
``````````

</details>

<details>
<summary>
View the diff from darker here.
</summary>

``````````diff
--- message-converter.py	2024-11-19 20:36:00.000000 +0000
+++ message-converter.py	2024-11-19 20:52:14.308621 +0000
@@ -19,27 +19,27 @@
 from libomputils import ScriptError, error
 
 
 class TargetPlatform:
     """Convenience class for handling the target platform for configuration/compilation"""
-    
+
     system_override = None
     """
     Target system name override by the user.
     It follows the conventions from https://docs.python.org/3/library/platform.html#platform.system
     """
 
     def set_system_override(override_system):
         """
-        Set a system override for the target. 
+        Set a system override for the target.
         Please follow the style from https://docs.python.org/3/library/platform.html#platform.system
         """
         TargetPlatform.system_override = override_system
 
     def system():
         """
-        Target System name. 
+        Target System name.
         It follows the conventions from https://docs.python.org/3/library/platform.html#platform.system
         """
         if TargetPlatform.system_override is None:
             return platform.system()
         return TargetPlatform.system_override
@@ -388,15 +388,15 @@
     )
     parser.add_argument(
         "--message", metavar="FILE", help="Generate message file named FILE"
     )
     parser.add_argument(
-        "--target-system-override", 
-        metavar="TARGET_SYSTEM_NAME", 
+        "--target-system-override",
+        metavar="TARGET_SYSTEM_NAME",
         help="Target System override.\n"
         "By default the target system is the host system\n"
-        "See possible values at https://docs.python.org/3/library/platform.html#platform.system"
+        "See possible values at https://docs.python.org/3/library/platform.html#platform.system",
     )
     parser.add_argument("inputfile")
     commandArgs = parser.parse_args()
 
     if commandArgs.lang_id:

``````````

</details>


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


More information about the Openmp-commits mailing list