[flang] Disable test on Windows

[klausler]

flang/test/Semantics/call40.f90 is behaving weirdly on Windows, perhaps due to a percent sign in a message that shouldn't be interpreted as a formatting character. Disable the test for now so that CI isn't producing confusing errors.

[llvmbot]

@llvm/pr-subscribers-flang-semantics

Author: Peter Klausler (klausler)

Changes

flang/test/Semantics/call40.f90 is behaving weirdly on Windows, perhaps due to a percent sign in a message that shouldn't be interpreted as a formatting character. Disable the test for now so that CI isn't producing confusing errors.

---

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

1 Files Affected:

• (modified) flang/test/Semantics/call40.f90 (+1)

diff --git a/flang/test/Semantics/call40.f90 b/flang/test/Semantics/call40.f90
index e240b5a432184..1a37ed5e2cfa4 100644
--- a/flang/test/Semantics/call40.f90
+++ b/flang/test/Semantics/call40.f90
@@ -1,4 +1,5 @@
 ! RUN: %python %S/test_errors.py %s %flang_fc1
+! XFAIL: system-windows
 ! %VAL en %REF legacy extension semantic tests.
 
 subroutine val_errors(array, string, polymorphic, derived)

[DavidSpickett]

From the CI:

Unexpectedly Passed Tests (1):
  Flang :: Semantics/call40.f90

If this test sometimes passes sometimes doesn't, it'll have to be marked unsupported instead.

[luporl]

From the CI:

Unexpectedly Passed Tests (1):
  Flang :: Semantics/call40.f90

If this test sometimes passes sometimes doesn't, it'll have to be marked unsupported instead.

Right, but after #94331, this test should always pass.

So it seems the CI failed because this PR already includes #94331, making the test unexpectedly pass.

[DavidSpickett]

Ok got it, so this PR is not needed now?

[luporl]

Ok got it, so this PR is not needed now?

Yes, unless this test starts failing again on Windows.