[llvm] [LIT][Cygwin] Mark `ulimit -n` unsupported for Cygwin (PR #164604)

via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 22 04:20:06 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-testing-tools

Author: Tomohiro Kashiwada (kikairoya)

<details>
<summary>Changes</summary>

Cygwin doesn't support `ulimit -n` because Windows doesn't provide such functionality.

---
Full diff: https://github.com/llvm/llvm-project/pull/164604.diff


2 Files Affected:

- (modified) llvm/utils/lit/lit/run.py (+2-2) 
- (modified) llvm/utils/lit/tests/shtest-ulimit.py (+1-1) 


``````````diff
diff --git a/llvm/utils/lit/lit/run.py b/llvm/utils/lit/lit/run.py
index 62070e824e87f..5830b21be6b0f 100644
--- a/llvm/utils/lit/lit/run.py
+++ b/llvm/utils/lit/lit/run.py
@@ -137,6 +137,6 @@ def _increase_process_limit(self):
                     "Raised process limit from %d to %d" % (soft_limit, desired_limit)
                 )
         except Exception as ex:
-            # Warn, unless this is Windows or z/OS, in which case this is expected.
-            if os.name != "nt" and platform.system() != "OS/390":
+            # Warn, unless this is Windows, z/OS, or Cygwin in which case this is expected.
+            if os.name != "nt" and platform.system() != "OS/390" and platform.sys.platform != "cygwin":
                 self.lit_config.warning("Failed to raise process limit: %s" % ex)
diff --git a/llvm/utils/lit/tests/shtest-ulimit.py b/llvm/utils/lit/tests/shtest-ulimit.py
index 9a8febd3333c4..c3d51067e8936 100644
--- a/llvm/utils/lit/tests/shtest-ulimit.py
+++ b/llvm/utils/lit/tests/shtest-ulimit.py
@@ -3,7 +3,7 @@
 # ulimit does not work on non-POSIX platforms.
 # Solaris for some reason does not respect ulimit -n, so mark it unsupported
 # as well.
-# UNSUPPORTED: system-windows, system-solaris
+# UNSUPPORTED: system-windows, system-cygwin, system-solaris
 
 # RUN: not %{lit} -a -v %{inputs}/shtest-ulimit --order=lexical | FileCheck %s
 

``````````

</details>


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


More information about the llvm-commits mailing list