EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land.
The `.fail.cpp` tests should use `// UNSUPPORTED: c++98, c++03, c++11, c++14` instead of `#error `. Other than that this LGTM. ================ Comment at: test/std/depr/depr.function.objects/depr.adaptors/depr.member.pointer.adaptors/const_mem_fun.cxx1z.fail.cpp:21 +#include "test_macros.h" +#if TEST_STD_VER <= 14 +#error ---------------- `// UNSUPPORTED: c++98, c++03, c++11, c++14` ================ Comment at: test/std/depr/depr.function.objects/depr.adaptors/depr.member.pointer.adaptors/const_mem_fun1.cxx1z.fail.cpp:21 +#include "test_macros.h" +#if TEST_STD_VER <= 14 +#error ---------------- `// UNSUPPORTED: c++98, c++03, c++11, c++14` ================ Comment at: test/std/depr/depr.function.objects/depr.adaptors/depr.member.pointer.adaptors/const_mem_fun1_ref_t.cxx1z.fail.cpp:20 +#include "test_macros.h" +#if TEST_STD_VER <= 14 +#error ---------------- `// UNSUPPORTED: c++98, c++03, c++11, c++14` ================ Comment at: test/std/depr/depr.function.objects/depr.adaptors/depr.member.pointer.adaptors/const_mem_fun1_t.cxx1z.fail.cpp:20 +#include "test_macros.h" +#if TEST_STD_VER <= 14 +#error ---------------- `// UNSUPPORTED: c++98, c++03, c++11, c++14` ================ Comment at: test/std/depr/depr.function.objects/depr.adaptors/depr.member.pointer.adaptors/const_mem_fun_ref.cxx1z.fail.cpp:21 +#include "test_macros.h" +#if TEST_STD_VER <= 14 +#error ---------------- `// UNSUPPORTED: c++98, c++03, c++11, c++14` ================ Comment at: test/std/depr/depr.function.objects/depr.adaptors/depr.member.pointer.adaptors/const_mem_fun_ref_t.cxx1z.fail.cpp:20 +#include "test_macros.h" +#if TEST_STD_VER <= 14 +#error ---------------- `// UNSUPPORTED: c++98, c++03, c++11, c++14` ================ Comment at: test/std/depr/depr.function.objects/depr.adaptors/depr.member.pointer.adaptors/const_mem_fun_t.cxx1z.fail.cpp:20 +#include "test_macros.h" +#if TEST_STD_VER <= 14 +#error ---------------- `// UNSUPPORTED: c++98, c++03, c++11, c++14` https://reviews.llvm.org/D31769 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
