PR libstdc++/86398 fix std::is_trivially_constructible regression

The intrinsic doesn't check for allowed conversions between scalar
types, so restore the std::is_constructible check.

Also make some trivial whitespace changes.

	PR libstdc++/86398
	* include/std/type_traits (is_trivially_constructible): Check
	is_constructible before __is_trivially_constructible.
	* testsuite/20_util/is_trivially_constructible/value.cc: Add more
	tests, including negative cases.
	* testsuite/20_util/make_signed/requirements/typedefs_neg.cc: Use
	zero for dg-error lineno.
	* testsuite/20_util/make_unsigned/requirements/typedefs_neg.cc:
	Likewise.

From-SVN: r262379

libstdc++-v3/ChangeLog

 2018-07-04  Jonathan Wakely  <jwakely@redhat.com>
 
+	PR libstdc++/86398
+	* include/std/type_traits (is_trivially_constructible): Check
+	is_constructible before __is_trivially_constructible.
+	* testsuite/20_util/is_trivially_constructible/value.cc: Add more
+	tests, including negative cases.
+	* testsuite/20_util/make_signed/requirements/typedefs_neg.cc: Use
+	zero for dg-error lineno.
+	* testsuite/20_util/make_unsigned/requirements/typedefs_neg.cc:
+	Likewise.
+
 	* include/std/bit (__rotl, __rotr): Avoid branch.
 	(_If_is_unsigned_integer): Use remove_cv_t.
 	* testsuite/26_numerics/bit/bitops.count/popcount.cc: New.

libstdc++-v3/include/std/type_traits

@@ -1136,7 +1136,8 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
   /// is_trivially_constructible
   template<typename _Tp, typename... _Args>
     struct is_trivially_constructible
-    : public __bool_constant<__is_trivially_constructible(_Tp, _Args...)>
+    : public __and_<is_constructible<_Tp, _Args...>, __bool_constant<
+		      __is_trivially_constructible(_Tp, _Args...)>>::type
     { };
 
   /// is_trivially_default_constructible

libstdc++-v3/testsuite/20_util/is_trivially_constructible/value.cc

@@ -55,6 +55,24 @@ void test01()
   static_assert(test_property<is_trivially_constructible,
 		int, const int&>(true), "");
   static_assert(test_property<is_trivially_constructible,
+		int, void*>(false), "PR 86398");
+  static_assert(test_property<is_trivially_constructible,
+		int, int*>(false), "PR 86398");
+  static_assert(test_property<is_trivially_constructible,
+		int, const int*>(false), "PR 86398");
+  static_assert(test_property<is_trivially_constructible,
+		int*, void*>(false), "PR 86398");
+  static_assert(test_property<is_trivially_constructible,
+		int*, const int*>(false), "PR 86398");
+  static_assert(test_property<is_trivially_constructible,
+		int&, const int>(false), "");
+  static_assert(test_property<is_trivially_constructible,
+		const int&, int>(true), "");
+  static_assert(test_property<is_trivially_constructible,
+		const int&, int&>(true), "");
+  static_assert(test_property<is_trivially_constructible,
+		const int*, int*>(true), "");
+  static_assert(test_property<is_trivially_constructible,
 		PolymorphicClass>(false), "");
   static_assert(test_property<is_trivially_constructible,
 		PolymorphicClass, PolymorphicClass>(false), "");
@@ -162,6 +180,4 @@ void test01()
 		MoveOnly, const MoveOnly&>(false), "");
   static_assert(test_property<is_trivially_constructible,
 		MoveOnly2>(false), "");
-
-
 }

libstdc++-v3/testsuite/20_util/make_signed/requirements/typedefs_neg.cc

@@ -47,4 +47,4 @@ void test01()
 // { dg-error "required from here" "" { target *-*-* } 39 }
 // { dg-error "required from here" "" { target *-*-* } 41 }
 
-// { dg-error "invalid use of incomplete type" "" { target *-*-* } 1825 }
+// { dg-error "invalid use of incomplete type" "" { target *-*-* } 0 }

libstdc++-v3/testsuite/20_util/make_unsigned/requirements/typedefs_neg.cc

@@ -47,5 +47,4 @@ void test01()
 // { dg-error "required from here" "" { target *-*-* } 39 }
 // { dg-error "required from here" "" { target *-*-* } 41 }
 
-// { dg-error "invalid use of incomplete type" "" { target *-*-* } 1708 }
-
+// { dg-error "invalid use of incomplete type" "" { target *-*-* } 0 }