[libunwind] [libc++][lit] Allow overriding the executor for tests (PR #66545)

Louis Dionne via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 25 09:05:22 PDT 2023


================
@@ -314,5 +317,14 @@ def getStdFlag(cfg, std):
             AddCompileFlag("-D_LIBCPP_REMOVE_TRANSITIVE_INCLUDES"),
         ],
     ),
+    Parameter(
+        name="executor",
+        type=str,
+        default=f"{sys.executable} {Path(__file__).resolve().parent.parent.parent / 'run.py'}",
+        help="Custom executor to use instead of the configured default.",
+        actions=lambda executor: [] if not executor else [
----------------
ldionne wrote:

There *has* to be an executor, so I think we should change to

```
actions=lambda executor: [AddSubstitution("%{executor}", executor)]
```

i.e. drop the `if`.

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


More information about the cfe-commits mailing list