[libcxx-commits] [PATCH] D99095: [libcxx] [test] Add XFAIL LIBCXX-WINDOWS-FIXME in 124 tests that fail in the CI configuration
Martin Storsjö via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
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
> 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
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the libcxx-commits