[libcxx-commits] [PATCH] D113348: [libc++] [test] Eliminate the libcpp-no-if-constexpr feature flag.

[Arthur O'Dwyer via Phabricator via libcxx-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rG9a3cb7346068: [libc++] [test] Eliminate the libcpp-no-if-constexpr feature flag. (authored by arthur.j.odwyer).

Changed prior to commit:
  https://reviews.llvm.org/D113348?vs=385289&id=385625#toc

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113348

Files:
  libcxx/test/libcxx/debug/containers/db_associative_container_tests.pass.cpp
  libcxx/test/libcxx/debug/containers/db_sequence_container_iterators.multithread.pass.cpp
  libcxx/test/libcxx/debug/containers/db_sequence_container_iterators.pass.cpp
  libcxx/test/libcxx/debug/containers/db_string.pass.cpp
  libcxx/test/libcxx/debug/containers/db_unord_container_tests.pass.cpp
  libcxx/test/libcxx/debug/db_string_view.pass.cpp
  libcxx/test/support/container_debug_tests.h
  libcxx/utils/libcxx/test/features.py


Index: libcxx/utils/libcxx/test/features.py
===================================================================
--- libcxx/utils/libcxx/test/features.py
+++ libcxx/utils/libcxx/test/features.py
@@ -36,7 +36,6 @@
   Feature(name='-fsized-deallocation',          when=lambda cfg: hasCompileFlag(cfg, '-fsized-deallocation')),
   Feature(name='-faligned-allocation',          when=lambda cfg: hasCompileFlag(cfg, '-faligned-allocation')),
   Feature(name='fdelayed-template-parsing',     when=lambda cfg: hasCompileFlag(cfg, '-fdelayed-template-parsing')),
-  Feature(name='libcpp-no-if-constexpr',        when=lambda cfg: '__cpp_if_constexpr' not in featureTestMacros(cfg)),
   Feature(name='libcpp-no-structured-bindings', when=lambda cfg: '__cpp_structured_bindings' not in featureTestMacros(cfg)),
   Feature(name='libcpp-no-concepts',            when=lambda cfg: featureTestMacros(cfg).get('__cpp_concepts', 0) < 201907),
   Feature(name='has-fobjc-arc',                 when=lambda cfg: hasCompileFlag(cfg, '-xobjective-c++ -fobjc-arc') and
Index: libcxx/test/support/container_debug_tests.h
===================================================================
--- libcxx/test/support/container_debug_tests.h
+++ libcxx/test/support/container_debug_tests.h
@@ -33,11 +33,6 @@
 #error This header may only be used in C++17 and greater
 #endif
 
-#ifndef __cpp_if_constexpr
-#error These tests require if constexpr
-#endif
-
-
 namespace IteratorDebugChecks {
 
 enum ContainerType {
Index: libcxx/test/libcxx/debug/db_string_view.pass.cpp
===================================================================
--- libcxx/test/libcxx/debug/db_string_view.pass.cpp
+++ libcxx/test/libcxx/debug/db_string_view.pass.cpp
@@ -8,7 +8,6 @@
 
 // UNSUPPORTED: c++03, c++11, c++14
 // UNSUPPORTED: windows
-// UNSUPPORTED: libcpp-no-if-constexpr
 
 // ADDITIONAL_COMPILE_FLAGS: -D_LIBCPP_DEBUG=1
 // UNSUPPORTED: libcxx-no-debug-mode
Index: libcxx/test/libcxx/debug/containers/db_unord_container_tests.pass.cpp
===================================================================
--- libcxx/test/libcxx/debug/containers/db_unord_container_tests.pass.cpp
+++ libcxx/test/libcxx/debug/containers/db_unord_container_tests.pass.cpp
@@ -8,7 +8,6 @@
 
 // UNSUPPORTED: c++03, c++11, c++14
 // UNSUPPORTED: windows
-// UNSUPPORTED: libcpp-no-if-constexpr
 
 // ADDITIONAL_COMPILE_FLAGS: -D_LIBCPP_DEBUG=1
 // UNSUPPORTED: libcxx-no-debug-mode
Index: libcxx/test/libcxx/debug/containers/db_string.pass.cpp
===================================================================
--- libcxx/test/libcxx/debug/containers/db_string.pass.cpp
+++ libcxx/test/libcxx/debug/containers/db_string.pass.cpp
@@ -8,7 +8,6 @@
 
 // UNSUPPORTED: c++03, c++11, c++14
 // UNSUPPORTED: windows
-// UNSUPPORTED: libcpp-no-if-constexpr
 
 // ADDITIONAL_COMPILE_FLAGS: -D_LIBCPP_DEBUG=1
 // UNSUPPORTED: libcxx-no-debug-mode
Index: libcxx/test/libcxx/debug/containers/db_sequence_container_iterators.pass.cpp
===================================================================
--- libcxx/test/libcxx/debug/containers/db_sequence_container_iterators.pass.cpp
+++ libcxx/test/libcxx/debug/containers/db_sequence_container_iterators.pass.cpp
@@ -8,7 +8,6 @@
 
 // UNSUPPORTED: c++03, c++11, c++14
 // UNSUPPORTED: windows
-// UNSUPPORTED: libcpp-no-if-constexpr
 
 // ADDITIONAL_COMPILE_FLAGS: -D_LIBCPP_DEBUG=1
 // UNSUPPORTED: libcxx-no-debug-mode
Index: libcxx/test/libcxx/debug/containers/db_sequence_container_iterators.multithread.pass.cpp
===================================================================
--- libcxx/test/libcxx/debug/containers/db_sequence_container_iterators.multithread.pass.cpp
+++ libcxx/test/libcxx/debug/containers/db_sequence_container_iterators.multithread.pass.cpp
@@ -9,7 +9,6 @@
 // UNSUPPORTED: c++03, c++11, c++14
 // UNSUPPORTED: windows
 // UNSUPPORTED: libcpp-has-no-threads
-// UNSUPPORTED: libcpp-no-if-constexpr
 
 // ADDITIONAL_COMPILE_FLAGS: -D_LIBCPP_DEBUG=1
 // UNSUPPORTED: libcxx-no-debug-mode
Index: libcxx/test/libcxx/debug/containers/db_associative_container_tests.pass.cpp
===================================================================
--- libcxx/test/libcxx/debug/containers/db_associative_container_tests.pass.cpp
+++ libcxx/test/libcxx/debug/containers/db_associative_container_tests.pass.cpp
@@ -8,7 +8,6 @@
 
 // UNSUPPORTED: c++03, c++11, c++14
 // UNSUPPORTED: windows
-// UNSUPPORTED: libcpp-no-if-constexpr
 
 // ADDITIONAL_COMPILE_FLAGS: -D_LIBCPP_DEBUG=1
 // UNSUPPORTED: libcxx-no-debug-mode


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D113348.385625.patch
Type: text/x-patch
Size: 4515 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20211108/8d38d166/attachment-0001.bin>