[libcxx-commits] [PATCH] D123145: [libcxx] [test] Remove LIBCXX-WINDOWS-FIXME

Martin Storsjö via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Apr 5 10:10:39 PDT 2022


mstorsjo created this revision.
mstorsjo added reviewers: ldionne, Mordante.
Herald added a subscriber: arichardson.
Herald added a project: All.
mstorsjo requested review of this revision.
Herald added a project: libc++.
Herald added a reviewer: libc++.

There are no such markings left - all of them have been fixed or
analyzed.

This closes llvm.org/PR32730 (github issue #32077).


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D123145

Files:
  libcxx/test/configs/llvm-libc++-mingw.cfg.in
  libcxx/test/configs/llvm-libc++-shared-clangcl.cfg.in
  libcxx/test/configs/llvm-libc++-static-clangcl.cfg.in
  libcxx/utils/libcxx/test/config.py


Index: libcxx/utils/libcxx/test/config.py
===================================================================
--- libcxx/utils/libcxx/test/config.py
+++ libcxx/utils/libcxx/test/config.py
@@ -133,7 +133,6 @@
         self.configure_env()
         self.configure_coverage()
         self.configure_substitutions()
-        self.configure_features()
 
         libcxx.test.newconfig.configure(
             libcxx.test.params.DEFAULT_PARAMETERS,
@@ -222,16 +221,6 @@
             else:
                 self.libcxx_obj_root = self.project_obj_root
 
-    def configure_features(self):
-        if self.target_info.is_windows():
-            if self.cxx_stdlib_under_test == 'libc++':
-                # LIBCXX-WINDOWS-FIXME is the feature name used to XFAIL the
-                # initial Windows failures until they can be properly diagnosed
-                # and fixed. This allows easier detection of new test failures
-                # and regressions. Note: New failures should not be suppressed
-                # using this feature. (Also see llvm.org/PR32730)
-                self.config.available_features.add('LIBCXX-WINDOWS-FIXME')
-
     def configure_compile_flags(self):
         self.configure_default_compile_flags()
         # Configure extra flags
Index: libcxx/test/configs/llvm-libc++-static-clangcl.cfg.in
===================================================================
--- libcxx/test/configs/llvm-libc++-static-clangcl.cfg.in
+++ libcxx/test/configs/llvm-libc++-static-clangcl.cfg.in
@@ -14,13 +14,6 @@
     '%{executor} --execdir %T --env PATH=%{lib} -- '
 ))
 
-# LIBCXX-WINDOWS-FIXME is the feature name used to XFAIL the
-# initial Windows failures until they can be properly diagnosed
-# and fixed. This allows easier detection of new test failures
-# and regressions. Note: New failures should not be suppressed
-# using this feature. (Also see llvm.org/PR32730)
-config.available_features.add('LIBCXX-WINDOWS-FIXME')
-
 import os, site
 site.addsitedir(os.path.join('@LIBCXX_SOURCE_DIR@', 'utils'))
 import libcxx.test.params, libcxx.test.newconfig
Index: libcxx/test/configs/llvm-libc++-shared-clangcl.cfg.in
===================================================================
--- libcxx/test/configs/llvm-libc++-shared-clangcl.cfg.in
+++ libcxx/test/configs/llvm-libc++-shared-clangcl.cfg.in
@@ -14,13 +14,6 @@
     '%{executor} --execdir %T --env PATH=%{lib} -- '
 ))
 
-# LIBCXX-WINDOWS-FIXME is the feature name used to XFAIL the
-# initial Windows failures until they can be properly diagnosed
-# and fixed. This allows easier detection of new test failures
-# and regressions. Note: New failures should not be suppressed
-# using this feature. (Also see llvm.org/PR32730)
-config.available_features.add('LIBCXX-WINDOWS-FIXME')
-
 import os, site
 site.addsitedir(os.path.join('@LIBCXX_SOURCE_DIR@', 'utils'))
 import libcxx.test.params, libcxx.test.newconfig
Index: libcxx/test/configs/llvm-libc++-mingw.cfg.in
===================================================================
--- libcxx/test/configs/llvm-libc++-mingw.cfg.in
+++ libcxx/test/configs/llvm-libc++-mingw.cfg.in
@@ -14,13 +14,6 @@
     '%{executor} --execdir %T --env PATH=%{lib} -- '
 ))
 
-# LIBCXX-WINDOWS-FIXME is the feature name used to XFAIL the
-# initial Windows failures until they can be properly diagnosed
-# and fixed. This allows easier detection of new test failures
-# and regressions. Note: New failures should not be suppressed
-# using this feature. (Also see llvm.org/PR32730)
-config.available_features.add('LIBCXX-WINDOWS-FIXME')
-
 import os, site
 site.addsitedir(os.path.join('@LIBCXX_SOURCE_DIR@', 'utils'))
 import libcxx.test.params, libcxx.test.newconfig


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D123145.420570.patch
Type: text/x-patch
Size: 3691 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220405/1e4bff94/attachment.bin>


More information about the libcxx-commits mailing list