[libcxx-commits] [PATCH] D99095: [libcxx] [test] Add XFAIL LIBCXX-WINDOWS-FIXME in 124 tests that fail in the CI configuration

Wed Mar 24 12:15:07 PDT 2021

mstorsjo added a comment.

In D99095#2648525 <https://reviews.llvm.org/D99095#2648525>, @Mordante wrote:

> I'm not sure how temporary this solution is, but there's an issue with the feature-test macro script after this commit. IMO if it's very temporary we don't need to take action otherwise I think we should look at a fix.

In very general terms, some of the added xfails won't stay for long, while others certainly will stick around for some time. (The fix then either being some change making the tests pass, or changed into `UNSUPPORTED: windows` or `XFAIL: windows` after analysis and explanation why.) It partially comes down to review throughput too ;-)

================
Comment at: libcxx/test/std/language.support/support.limits/support.limits.general/complex.version.pass.cpp:16

+// XFAIL: LIBCXX-WINDOWS-FIXME
+
----------------
Mordante wrote:
> This file and the other `*.version.pass.cpp` are automatically generated by `utils/generate_feature_test_macro_components.py`. If a developer runs this script these changes will accidentally be reverted. I'm not sure what would be the best fix for now.
Those particular xfails can be removed with D99213

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99095/new/

https://reviews.llvm.org/D99095