From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1984) id 19ED03A7482C; Fri, 17 Jul 2020 14:40:23 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 19ED03A7482C DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1594996823; bh=8S1OLstWJcJ3688K3aNU070LjS7v8kCm8z5ypiczgfw=; h=From:To:Subject:Date:From; b=mvTWF1ntmsIt89B4TAH11tisJn49PwQBRbhsr61EkNRLcf20JjzBF+6yc82RNJkEO 9A46QpI4jjnu+ygBS3LeUsRQ8BIh/OI6+qGbHpgfXifXqQ9Q7vzM/TpL6BYiZdmY14 RvkPezDuADPRb60Bg6gA1QfwquRdgkFgov5Q5E9I= Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Tamar Christina To: gcc-cvs@gcc.gnu.org, libstdc++-cvs@gcc.gnu.org Subject: [gcc(refs/vendors/ARM/heads/arm-struct-reorg-wip)] libstdc++: Define __cpp_lib_execution feature test macro X-Act-Checkin: gcc X-Git-Author: Jonathan Wakely X-Git-Refname: refs/vendors/ARM/heads/arm-struct-reorg-wip X-Git-Oldrev: e851aa1703c3d911720d8f9bee68a521abe588f7 X-Git-Newrev: aac39307e8f589dba231083a3e9aa3d0ff302837 Message-Id: <20200717144023.19ED03A7482C@sourceware.org> Date: Fri, 17 Jul 2020 14:40:23 +0000 (GMT) X-BeenThere: libstdc++-cvs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libstdc++-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 17 Jul 2020 14:40:23 -0000 https://gcc.gnu.org/g:aac39307e8f589dba231083a3e9aa3d0ff302837 commit aac39307e8f589dba231083a3e9aa3d0ff302837 Author: Jonathan Wakely Date: Wed Apr 22 22:54:35 2020 +0100 libstdc++: Define __cpp_lib_execution feature test macro This macro has never been defined by libstdc++, despite supporting the parallel algorithms. It should have a different value for C++17 and C++20, because P1001R2 should not be supported in C++17, but unsequenced_policy is defined for C++17 (see PR p4702). * include/std/execution (__cpp_lib_execution): Define to indicate support for P0024R2 and P1001R2. * include/std/version (__cpp_lib_execution): Define. * testsuite/25_algorithms/pstl/feature_test.cc: Only test macro defined by , move other tests to new tests ... * testsuite/25_algorithms/pstl/feature_test-2.cc: New test. * testsuite/25_algorithms/pstl/feature_test-3.cc: New test. * testsuite/25_algorithms/pstl/feature_test-4.cc: New test. * testsuite/25_algorithms/pstl/feature_test-5.cc: New test. Diff: --- libstdc++-v3/ChangeLog | 10 +++++++ libstdc++-v3/include/std/execution | 1 + libstdc++-v3/include/std/version | 2 ++ .../testsuite/25_algorithms/pstl/feature_test-2.cc | 27 +++++++++++++++++ .../testsuite/25_algorithms/pstl/feature_test-3.cc | 34 ++++++++++++++++++++++ .../testsuite/25_algorithms/pstl/feature_test-4.cc | 33 +++++++++++++++++++++ .../testsuite/25_algorithms/pstl/feature_test-5.cc | 28 ++++++++++++++++++ .../testsuite/25_algorithms/pstl/feature_test.cc | 27 ++--------------- 8 files changed, 137 insertions(+), 25 deletions(-) diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index cce254968fb..fb0c1acd268 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,5 +1,15 @@ 2020-04-22 Jonathan Wakely + * include/std/execution (__cpp_lib_execution): Define to indicate + support for P0024R2 and P1001R2. + * include/std/version (__cpp_lib_execution): Define. + * testsuite/25_algorithms/pstl/feature_test.cc: Only test macro + defined by , move other tests to new tests ... + * testsuite/25_algorithms/pstl/feature_test-2.cc: New test. + * testsuite/25_algorithms/pstl/feature_test-3.cc: New test. + * testsuite/25_algorithms/pstl/feature_test-4.cc: New test. + * testsuite/25_algorithms/pstl/feature_test-5.cc: New test. + * include/bits/stl_iterator.h (__cpp_lib_array_constexpr): Define different values for C++17 and C++20, to indicate different feature sets. Update value for C++20 to indicate P1032R1 support. diff --git a/libstdc++-v3/include/std/execution b/libstdc++-v3/include/std/execution index ac78af8eac3..cdd852deb14 100644 --- a/libstdc++-v3/include/std/execution +++ b/libstdc++-v3/include/std/execution @@ -50,6 +50,7 @@ // Feature test macro for parallel algorithms # define __cpp_lib_parallel_algorithm 201603L +# define __cpp_lib_execution 201902L #endif // C++17 diff --git a/libstdc++-v3/include/std/version b/libstdc++-v3/include/std/version index 57a05259d98..3024f1caf8b 100644 --- a/libstdc++-v3/include/std/version +++ b/libstdc++-v3/include/std/version @@ -127,6 +127,7 @@ #define __cpp_lib_clamp 201603 #define __cpp_lib_constexpr_char_traits 201611 #define __cpp_lib_enable_shared_from_this 201603 +#define __cpp_lib_execution 201902L // FIXME: should be 201603L #define __cpp_lib_filesystem 201703 #define __cpp_lib_gcd 201606 #define __cpp_lib_gcd_lcm 201606 @@ -188,6 +189,7 @@ #define __cpp_lib_array_constexpr 201811L #define __cpp_lib_assume_aligned 201811L #define __cpp_lib_bind_front 201907L +// FIXME: #define __cpp_lib_execution 201902L #define __cpp_lib_integer_comparison_functions 202002L #define __cpp_lib_constexpr_algorithms 201806L #define __cpp_lib_constexpr_complex 201711L diff --git a/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-2.cc b/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-2.cc new file mode 100644 index 00000000000..3e74f89bc41 --- /dev/null +++ b/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-2.cc @@ -0,0 +1,27 @@ +// Copyright (C) 2020 Free Software Foundation, Inc. +// +// This file is part of the GNU ISO C++ Library. This library is free +// software; you can redistribute it and/or modify it under the +// terms of the GNU General Public License as published by the +// Free Software Foundation; either version 3, or (at your option) +// any later version. + +// This library is distributed in the hope that it will be useful, +// but WITHOUT ANY WARRANTY; without even the implied warranty of +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +// GNU General Public License for more details. + +// You should have received a copy of the GNU General Public License along +// with this library; see the file COPYING3. If not see +// . + +// { dg-options "-std=gnu++17" } +// { dg-do preprocess { target c++17 } } + +#include + +#ifndef __cpp_lib_parallel_algorithm +# error "Feature-test macro for parallel algorithms missing in " +#elif __cpp_lib_parallel_algorithm != 201603L +# error "Feature-test macro for parallel algorithms has wrong value in " +#endif diff --git a/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-3.cc b/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-3.cc new file mode 100644 index 00000000000..7693fe03548 --- /dev/null +++ b/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-3.cc @@ -0,0 +1,34 @@ +// Copyright (C) 2020 Free Software Foundation, Inc. +// +// This file is part of the GNU ISO C++ Library. This library is free +// software; you can redistribute it and/or modify it under the +// terms of the GNU General Public License as published by the +// Free Software Foundation; either version 3, or (at your option) +// any later version. + +// This library is distributed in the hope that it will be useful, +// but WITHOUT ANY WARRANTY; without even the implied warranty of +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +// GNU General Public License for more details. + +// You should have received a copy of the GNU General Public License along +// with this library; see the file COPYING3. If not see +// . + +// { dg-options "-std=gnu++17" } +// { dg-do preprocess { target c++17 } } + +#include + +#ifndef __cpp_lib_execution +# error "Feature-test macro for execution policies is missing in " +#elif __cpp_lib_execution != 201902L +# error "Feature-test macro for parallel algorithms has wrong value in " +#endif + +// Neither SD-6 nor C++20 requires this macro to be defined in . +#ifndef __cpp_lib_parallel_algorithm +# error "Feature-test macro for parallel algorithms missing in " +#elif __cpp_lib_parallel_algorithm != 201603L +# error "Feature-test macro for parallel algorithms has wrong value in " +#endif diff --git a/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-4.cc b/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-4.cc new file mode 100644 index 00000000000..e7bbf920499 --- /dev/null +++ b/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-4.cc @@ -0,0 +1,33 @@ +// Copyright (C) 2020 Free Software Foundation, Inc. +// +// This file is part of the GNU ISO C++ Library. This library is free +// software; you can redistribute it and/or modify it under the +// terms of the GNU General Public License as published by the +// Free Software Foundation; either version 3, or (at your option) +// any later version. + +// This library is distributed in the hope that it will be useful, +// but WITHOUT ANY WARRANTY; without even the implied warranty of +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +// GNU General Public License for more details. + +// You should have received a copy of the GNU General Public License along +// with this library; see the file COPYING3. If not see +// . + +// { dg-options "-std=gnu++17" } +// { dg-do preprocess { target c++17 } } + +#include + +#ifndef __cpp_lib_parallel_algorithm +# error "Feature-test macro for parallel algorithms missing in " +#elif __cpp_lib_parallel_algorithm != 201603L +# error "Feature-test macro for parallel algorithms has wrong value in " +#endif + +#ifndef __cpp_lib_execution +# error "Feature-test macro for execution policies is missing in " +#elif __cpp_lib_execution != 201902L +# error "Feature-test macro for parallel algorithms has wrong value in " +#endif diff --git a/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-5.cc b/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-5.cc new file mode 100644 index 00000000000..2d991958e75 --- /dev/null +++ b/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test-5.cc @@ -0,0 +1,28 @@ +// Copyright (C) 2020 Free Software Foundation, Inc. +// +// This file is part of the GNU ISO C++ Library. This library is free +// software; you can redistribute it and/or modify it under the +// terms of the GNU General Public License as published by the +// Free Software Foundation; either version 3, or (at your option) +// any later version. + +// This library is distributed in the hope that it will be useful, +// but WITHOUT ANY WARRANTY; without even the implied warranty of +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +// GNU General Public License for more details. + +// You should have received a copy of the GNU General Public License along +// with this library; see the file COPYING3. If not see +// . + +// { dg-options "-std=gnu++17" } +// { dg-do preprocess { target c++17 } } + +#include + +// Neither SD-6 nor C++20 requires this macro to be defined in . +#ifndef __cpp_lib_parallel_algorithm +# error "Feature-test macro for parallel algorithms missing in " +#elif __cpp_lib_parallel_algorithm != 201603L +# error "Feature-test macro for parallel algorithms has wrong value in " +#endif diff --git a/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test.cc b/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test.cc index ce41f538c83..c3a9be5e45a 100644 --- a/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test.cc +++ b/libstdc++-v3/testsuite/25_algorithms/pstl/feature_test.cc @@ -19,32 +19,9 @@ // { dg-do preprocess { target c++17 } } #include + #ifndef __cpp_lib_parallel_algorithm -# error "Feature-test macro for parallel algorithms missing" +# error "Feature-test macro for parallel algorithms missing in " #elif __cpp_lib_parallel_algorithm != 201603L # error "Feature-test macro for parallel algorithms has wrong value in " #endif - -#include -#if __cpp_lib_parallel_algorithm != 201603L -# error "Feature-test macro for parallel algorithms has wrong value in " -#endif - -#include -#if __cpp_lib_parallel_algorithm != 201603L -# error "Feature-test macro for parallel algorithms has wrong value in " -#endif - -// The N4810 draft does not require the macro to be defined in . -#include -#if __cpp_lib_parallel_algorithm != 201603L -# error "Feature-test macro for parallel algorithms has wrong value in " -#endif - -// The N4810 draft does not require the macro to be defined in . -// Include this last, because it will trigger the inclusion of TBB headers, -// which then include , so we need to have already checked . -#include -#if __cpp_lib_parallel_algorithm != 201603L -# error "Feature-test macro for parallel algorithms has wrong value in " -#endif