]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
Remove redundant static assertions in [meta.unary.prop] traits
authorJonathan Wakely <jwakely@redhat.com>
Thu, 6 Jun 2019 12:13:47 +0000 (13:13 +0100)
committerJonathan Wakely <redi@gcc.gnu.org>
Thu, 6 Jun 2019 12:13:47 +0000 (13:13 +0100)
The type property predicates that are implemented by a compiler builtin
already do the right checks in the compiler. The checks for complete
type or unbounded arrays were wrong for these types anyway.

* include/std/type_traits (is_empty, is_polymorphic, is_final)
(is_abstract, is_aggregate): Remove static_assert.
* testsuite/20_util/is_abstract/incomplete_neg.cc: Check for error
from builtin only.
* testsuite/20_util/is_aggregate/incomplete_neg.cc: Likewise. Add
missing -std=gnu++17 option.
* testsuite/20_util/is_empty/incomplete_neg.cc: New test.
* testsuite/20_util/is_final/incomplete_neg.cc: New test.
* testsuite/20_util/is_polymorphic/incomplete_neg.cc: Check for error
from builtin only.

From-SVN: r272000

libstdc++-v3/ChangeLog
libstdc++-v3/include/std/type_traits
libstdc++-v3/testsuite/20_util/is_abstract/incomplete_neg.cc
libstdc++-v3/testsuite/20_util/is_aggregate/incomplete_neg.cc
libstdc++-v3/testsuite/20_util/is_empty/incomplete_neg.cc [new file with mode: 0644]
libstdc++-v3/testsuite/20_util/is_final/incomplete_neg.cc [new file with mode: 0644]
libstdc++-v3/testsuite/20_util/is_polymorphic/incomplete_neg.cc

index 283c6b9e057eb195d65aaa767b7dc11841c559e4..3dd82b637ff6c5b7fcb26f34ce0b8164aa0195a9 100644 (file)
@@ -1,5 +1,16 @@
 2019-06-06  Jonathan Wakely  <jwakely@redhat.com>
 
+       * include/std/type_traits (is_empty, is_polymorphic, is_final)
+       (is_abstract, is_aggregate): Remove static_assert.
+       * testsuite/20_util/is_abstract/incomplete_neg.cc: Check for error
+       from builtin only.
+       * testsuite/20_util/is_aggregate/incomplete_neg.cc: Likewise. Add
+       missing -std=gnu++17 option.
+       * testsuite/20_util/is_empty/incomplete_neg.cc: New test.
+       * testsuite/20_util/is_final/incomplete_neg.cc: New test.
+       * testsuite/20_util/is_polymorphic/incomplete_neg.cc: Check for error
+       from builtin only.
+
        * testsuite/18_support/set_terminate.cc: Do not run for C++98 mode.
        * testsuite/18_support/set_unexpected.cc: Likewise.
        * testsuite/20_util/is_nothrow_invocable/value.cc: Test converting to
index 78a113af415a391344853cf262a07530504affd4..e53d3c8d53543ab0762e5f83ad3af3720ea66655 100644 (file)
@@ -746,19 +746,13 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
   template<typename _Tp>
     struct is_empty
     : public integral_constant<bool, __is_empty(_Tp)>
-    {
-      static_assert(std::__is_complete_or_unbounded(__type_identity<_Tp>{}),
-       "template argument must be a complete class or an unbounded array");
-    };
+    { };
 
   /// is_polymorphic
   template<typename _Tp>
     struct is_polymorphic
     : public integral_constant<bool, __is_polymorphic(_Tp)>
-    {
-      static_assert(std::__is_complete_or_unbounded(__type_identity<_Tp>{}),
-       "template argument must be a complete class or an unbounded array");
-    };
+    { };
 
 #if __cplusplus >= 201402L
 #define __cpp_lib_is_final 201402L
@@ -766,20 +760,14 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
   template<typename _Tp>
     struct is_final
     : public integral_constant<bool, __is_final(_Tp)>
-    {
-      static_assert(std::__is_complete_or_unbounded(__type_identity<_Tp>{}),
-       "template argument must be a complete class or an unbounded array");
-    };
+    { };
 #endif
 
   /// is_abstract
   template<typename _Tp>
     struct is_abstract
     : public integral_constant<bool, __is_abstract(_Tp)>
-    {
-      static_assert(std::__is_complete_or_unbounded(__type_identity<_Tp>{}),
-       "template argument must be a complete class or an unbounded array");
-    };
+    { };
 
   template<typename _Tp,
           bool = is_arithmetic<_Tp>::value>
@@ -3174,10 +3162,7 @@ template <typename _From, typename _To>
   template<typename _Tp>
     struct is_aggregate
     : bool_constant<__is_aggregate(remove_cv_t<_Tp>)>
-    {
-      static_assert(std::__is_complete_or_unbounded(__type_identity<_Tp>{}),
-       "template argument must be a complete class or an unbounded array");
-    };
+    { };
 
   /// is_aggregate_v
   template<typename _Tp>
index 94f4ecd6000b8e2a948e88ae402adf9f3d79c7c4..a2a73d01a066ab1e91e2fed1cf4683398cc643fe 100644 (file)
@@ -1,7 +1,5 @@
 // { dg-do compile { target c++11 } }
-// { dg-prune-output "invalid use of incomplete type" }
-// { dg-prune-output "must be a complete" }
-//
+
 // Copyright (C) 2019 Free Software Foundation, Inc.
 //
 // This file is part of the GNU ISO C++ Library.  This library is free
@@ -19,6 +17,9 @@
 // with this library; see the file COPYING3.  If not see
 // <http://www.gnu.org/licenses/>.
 
+// Expect the compiler builtin to do the completeness check.
+// { dg-error "incomplete type" "" { target *-*-* } 0 }
+
 #include <type_traits>
 
 class X;
index 8a3dd551cbb856df5a213fc265177612e6bac176..eff3f64c476022f350747c1251eb950f2ace8e8a 100644 (file)
@@ -1,5 +1,6 @@
+// { dg-options "-std=gnu++17" }
 // { dg-do compile { target c++17 } }
-//
+
 // Copyright (C) 2019 Free Software Foundation, Inc.
 //
 // This file is part of the GNU ISO C++ Library.  This library is free
@@ -17,7 +18,8 @@
 // with this library; see the file COPYING3.  If not see
 // <http://www.gnu.org/licenses/>.
 
-// { dg-error "must be a complete class" "" { target *-*-* } 0 }
+// Expect the compiler builtin to do the completeness check.
+// { dg-error "incomplete type" "" { target *-*-* } 0 }
 
 #include <type_traits>
 
diff --git a/libstdc++-v3/testsuite/20_util/is_empty/incomplete_neg.cc b/libstdc++-v3/testsuite/20_util/is_empty/incomplete_neg.cc
new file mode 100644 (file)
index 0000000..3bacefb
--- /dev/null
@@ -0,0 +1,30 @@
+// { dg-do compile { target c++11 } }
+
+// Copyright (C) 2019 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
+// <http://www.gnu.org/licenses/>.
+
+// Expect the compiler builtin to do the completeness check.
+// { dg-error "incomplete type" "" { target *-*-* } 0 }
+
+#include <type_traits>
+
+class X;
+
+void test01()
+{
+  std::is_empty<X>();          // { dg-error "required from here" }
+}
diff --git a/libstdc++-v3/testsuite/20_util/is_final/incomplete_neg.cc b/libstdc++-v3/testsuite/20_util/is_final/incomplete_neg.cc
new file mode 100644 (file)
index 0000000..0727c09
--- /dev/null
@@ -0,0 +1,30 @@
+// { dg-do compile { target c++14 } }
+
+// Copyright (C) 2019 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
+// <http://www.gnu.org/licenses/>.
+
+// Expect the compiler builtin to do the completeness check.
+// { dg-error "incomplete type" "" { target *-*-* } 0 }
+
+#include <type_traits>
+
+class X;
+
+void test01()
+{
+  std::is_final<X>();          // { dg-error "required from here" }
+}
index 8cd1b402a41fba6a34bef339b15b9fdeaefaf180..618c4fa7a9a1b99f11b1d51a2849eb3a082bb000 100644 (file)
@@ -1,7 +1,5 @@
 // { dg-do compile { target c++11 } }
-// { dg-prune-output "invalid use of incomplete type" }
-// { dg-prune-output "must be a complete" }
-//
+
 // Copyright (C) 2019 Free Software Foundation, Inc.
 //
 // This file is part of the GNU ISO C++ Library.  This library is free
@@ -19,6 +17,9 @@
 // with this library; see the file COPYING3.  If not see
 // <http://www.gnu.org/licenses/>.
 
+// Expect the compiler builtin to do the completeness check.
+// { dg-error "incomplete type" "" { target *-*-* } 0 }
+
 #include <type_traits>
 
 class X;