[llvm] [lit][NFC] Avoid unintended -EMPTY suffix in check prefix (PR #99690)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 19 12:39:30 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-testing-tools
Author: Paul Kirth (ilovepi)
<details>
<summary>Changes</summary>
FileCheck has special handline for the `-EMPTY` suffix, that should
match empty lines. Overloading the suffix can be a source of confusion
when reading tests. Additionally, the current implementation seems to
match the following expressions, which appears to be a bug in FileCheck.
---
Full diff: https://github.com/llvm/llvm-project/pull/99690.diff
1 Files Affected:
- (modified) llvm/utils/lit/tests/Inputs/shtest-env/env-calls-env.txt (+3-3)
``````````diff
diff --git a/llvm/utils/lit/tests/Inputs/shtest-env/env-calls-env.txt b/llvm/utils/lit/tests/Inputs/shtest-env/env-calls-env.txt
index e4997b1250ed3..26150c413dc03 100644
--- a/llvm/utils/lit/tests/Inputs/shtest-env/env-calls-env.txt
+++ b/llvm/utils/lit/tests/Inputs/shtest-env/env-calls-env.txt
@@ -1,10 +1,10 @@
# Check that internal env can call internal env.
# RUN: env env %{python} print_environment.py \
-# RUN: | FileCheck -check-prefix=CHECK-2-EMPTY %s
+# RUN: | FileCheck -check-prefix=CHECK-2-EMPTY-ARGS %s
#
-# CHECK-2-EMPTY: BAR = 2
-# CHECK-2-EMPTY: FOO = 1
+# CHECK-2-EMPTY-ARGS: BAR = 2
+# CHECK-2-EMPTY-ARGS: FOO = 1
# RUN: env FOO=2 env BAR=1 %{python} print_environment.py \
# RUN: | FileCheck -check-prefix=CHECK-2-VAL %s
``````````
</details>
https://github.com/llvm/llvm-project/pull/99690
More information about the llvm-commits
mailing list