Hi again,
On 06/11/2017 12:40, Paolo Carlini wrote:
Hi,
On 06/11/2017 11:41, Martin Liška wrote:
Sorry for that. It's related to x86 target specific test-cases. I'll
prepare patch as soon as possible.
Ok, thanks.
Note, I'm seeing a few - expected, give the enabled warning - fails in
the libstdc++-v3 testsuite too. Eg:
FAIL: 20_util/optional/cons/deduction.cc (test for excess errors)
FAIL: 20_util/pair/cons/deduction.cc (test for excess errors)
FAIL: 20_util/pair/traits.cc (test for excess errors)
FAIL: 20_util/tuple/cons/deduction.cc (test for excess errors)
For the library I'm finishing testing the below.
Paolo.
////////////////////
2017-11-06 Paolo Carlini <paolo.carl...@oracle.com>
* testsuite/20_util/optional/cons/deduction.cc: Avoid -Wreturn-type
warnings.
* testsuite/20_util/pair/cons/deduction.cc: Likewise.
* testsuite/20_util/pair/traits.cc: Likewise.
* testsuite/20_util/tuple/cons/deduction.cc: Likewise.
* testsuite/20_util/variant/compile.cc: Likewise.
* testsuite/23_containers/map/modifiers/try_emplace/1.cc: Likewise.
* testsuite/23_containers/unordered_map/modifiers/try_emplace.cc:
Likewise.
Index: testsuite/20_util/optional/cons/deduction.cc
===================================================================
--- testsuite/20_util/optional/cons/deduction.cc (revision 254449)
+++ testsuite/20_util/optional/cons/deduction.cc (working copy)
@@ -24,8 +24,8 @@
struct MoveOnly
{
MoveOnly() = default;
- MoveOnly(MoveOnly&&) {}
- MoveOnly& operator=(MoveOnly&&) {}
+ MoveOnly(MoveOnly&&);
+ MoveOnly& operator=(MoveOnly&&);
};
int main()
Index: testsuite/20_util/pair/cons/deduction.cc
===================================================================
--- testsuite/20_util/pair/cons/deduction.cc (revision 254449)
+++ testsuite/20_util/pair/cons/deduction.cc (working copy)
@@ -30,8 +30,8 @@
struct MoveOnly
{
MoveOnly() = default;
- MoveOnly(MoveOnly&&) {}
- MoveOnly& operator=(MoveOnly&&) {}
+ MoveOnly(MoveOnly&&);
+ MoveOnly& operator=(MoveOnly&&);
};
void
Index: testsuite/20_util/pair/traits.cc
===================================================================
--- testsuite/20_util/pair/traits.cc (revision 254449)
+++ testsuite/20_util/pair/traits.cc (working copy)
@@ -31,8 +31,8 @@
struct ThrowingCopy
{
- ThrowingCopy(const ThrowingCopy&) {}
- ThrowingCopy& operator=(const ThrowingCopy&) {}
+ ThrowingCopy(const ThrowingCopy&);
+ ThrowingCopy& operator=(const ThrowingCopy&);
};
int main()
Index: testsuite/20_util/tuple/cons/deduction.cc
===================================================================
--- testsuite/20_util/tuple/cons/deduction.cc (revision 254442)
+++ testsuite/20_util/tuple/cons/deduction.cc (working copy)
@@ -30,8 +30,8 @@
struct MoveOnly
{
MoveOnly() = default;
- MoveOnly(MoveOnly&&) {}
- MoveOnly& operator=(MoveOnly&&) {}
+ MoveOnly(MoveOnly&&);
+ MoveOnly& operator=(MoveOnly&&);
};
void
Index: testsuite/20_util/variant/compile.cc
===================================================================
--- testsuite/20_util/variant/compile.cc (revision 254442)
+++ testsuite/20_util/variant/compile.cc (working copy)
@@ -481,37 +481,37 @@
static_assert(MA_VAL == is_trivially_move_assignable_v<variant<A>>, ""); \
}
TEST_TEMPLATE(=default, =default, =default, =default, =default, true,
true, true, true)
- TEST_TEMPLATE(=default, =default, =default, =default, {}, true,
true, true, false)
- TEST_TEMPLATE(=default, =default, =default, {}, =default, true,
true, false, true)
- TEST_TEMPLATE(=default, =default, =default, {}, {}, true,
true, false, false)
- TEST_TEMPLATE(=default, =default, {}, =default, =default, true,
false, true, true)
- TEST_TEMPLATE(=default, =default, {}, =default, {}, true,
false, true, false)
- TEST_TEMPLATE(=default, =default, {}, {}, =default, true,
false, false, true)
- TEST_TEMPLATE(=default, =default, {}, {}, {}, true,
false, false, false)
- TEST_TEMPLATE(=default, {}, =default, =default, =default, false,
true, true, true)
- TEST_TEMPLATE(=default, {}, =default, =default, {}, false,
true, true, false)
- TEST_TEMPLATE(=default, {}, =default, {}, =default, false,
true, false, true)
- TEST_TEMPLATE(=default, {}, =default, {}, {}, false,
true, false, false)
- TEST_TEMPLATE(=default, {}, {}, =default, =default, false,
false, true, true)
- TEST_TEMPLATE(=default, {}, {}, =default, {}, false,
false, true, false)
- TEST_TEMPLATE(=default, {}, {}, {}, =default, false,
false, false, true)
- TEST_TEMPLATE(=default, {}, {}, {}, {}, false,
false, false, false)
- TEST_TEMPLATE( {}, =default, =default, =default, =default, false,
false, false, false)
- TEST_TEMPLATE( {}, =default, =default, =default, {}, false,
false, false, false)
- TEST_TEMPLATE( {}, =default, =default, {}, =default, false,
false, false, false)
- TEST_TEMPLATE( {}, =default, =default, {}, {}, false,
false, false, false)
- TEST_TEMPLATE( {}, =default, {}, =default, =default, false,
false, false, false)
- TEST_TEMPLATE( {}, =default, {}, =default, {}, false,
false, false, false)
- TEST_TEMPLATE( {}, =default, {}, {}, =default, false,
false, false, false)
- TEST_TEMPLATE( {}, =default, {}, {}, {}, false,
false, false, false)
- TEST_TEMPLATE( {}, {}, =default, =default, =default, false,
false, false, false)
- TEST_TEMPLATE( {}, {}, =default, =default, {}, false,
false, false, false)
- TEST_TEMPLATE( {}, {}, =default, {}, =default, false,
false, false, false)
- TEST_TEMPLATE( {}, {}, =default, {}, {}, false,
false, false, false)
- TEST_TEMPLATE( {}, {}, {}, =default, =default, false,
false, false, false)
- TEST_TEMPLATE( {}, {}, {}, =default, {}, false,
false, false, false)
- TEST_TEMPLATE( {}, {}, {}, {}, =default, false,
false, false, false)
- TEST_TEMPLATE( {}, {}, {}, {}, {}, false,
false, false, false)
+ TEST_TEMPLATE(=default, =default, =default, =default, , true,
true, true, false)
+ TEST_TEMPLATE(=default, =default, =default, , =default, true,
true, false, true)
+ TEST_TEMPLATE(=default, =default, =default, , , true,
true, false, false)
+ TEST_TEMPLATE(=default, =default, , =default, =default, true,
false, true, true)
+ TEST_TEMPLATE(=default, =default, , =default, , true,
false, true, false)
+ TEST_TEMPLATE(=default, =default, , , =default, true,
false, false, true)
+ TEST_TEMPLATE(=default, =default, , , , true,
false, false, false)
+ TEST_TEMPLATE(=default, , =default, =default, =default, false,
true, true, true)
+ TEST_TEMPLATE(=default, , =default, =default, , false,
true, true, false)
+ TEST_TEMPLATE(=default, , =default, , =default, false,
true, false, true)
+ TEST_TEMPLATE(=default, , =default, , , false,
true, false, false)
+ TEST_TEMPLATE(=default, , , =default, =default, false,
false, true, true)
+ TEST_TEMPLATE(=default, , , =default, , false,
false, true, false)
+ TEST_TEMPLATE(=default, , , , =default, false,
false, false, true)
+ TEST_TEMPLATE(=default, , , , , false,
false, false, false)
+ TEST_TEMPLATE( , =default, =default, =default, =default, false,
false, false, false)
+ TEST_TEMPLATE( , =default, =default, =default, , false,
false, false, false)
+ TEST_TEMPLATE( , =default, =default, , =default, false,
false, false, false)
+ TEST_TEMPLATE( , =default, =default, , , false,
false, false, false)
+ TEST_TEMPLATE( , =default, , =default, =default, false,
false, false, false)
+ TEST_TEMPLATE( , =default, , =default, , false,
false, false, false)
+ TEST_TEMPLATE( , =default, , , =default, false,
false, false, false)
+ TEST_TEMPLATE( , =default, , , , false,
false, false, false)
+ TEST_TEMPLATE( , , =default, =default, =default, false,
false, false, false)
+ TEST_TEMPLATE( , , =default, =default, , false,
false, false, false)
+ TEST_TEMPLATE( , , =default, , =default, false,
false, false, false)
+ TEST_TEMPLATE( , , =default, , , false,
false, false, false)
+ TEST_TEMPLATE( , , , =default, =default, false,
false, false, false)
+ TEST_TEMPLATE( , , , =default, , false,
false, false, false)
+ TEST_TEMPLATE( , , , , =default, false,
false, false, false)
+ TEST_TEMPLATE( , , , , , false,
false, false, false)
#undef TEST_TEMPLATE
#define TEST_TEMPLATE(CC, MC, CA, MA) \
@@ -529,21 +529,21 @@
static_assert(!is_trivially_move_assignable_v<variant<AllDeleted, A>>,
""); \
}
TEST_TEMPLATE(=default, =default, =default, =default)
- TEST_TEMPLATE(=default, =default, =default, {})
- TEST_TEMPLATE(=default, =default, {}, =default)
- TEST_TEMPLATE(=default, =default, {}, {})
- TEST_TEMPLATE(=default, {}, =default, =default)
- TEST_TEMPLATE(=default, {}, =default, {})
- TEST_TEMPLATE(=default, {}, {}, =default)
- TEST_TEMPLATE(=default, {}, {}, {})
- TEST_TEMPLATE( {}, =default, =default, =default)
- TEST_TEMPLATE( {}, =default, =default, {})
- TEST_TEMPLATE( {}, =default, {}, =default)
- TEST_TEMPLATE( {}, =default, {}, {})
- TEST_TEMPLATE( {}, {}, =default, =default)
- TEST_TEMPLATE( {}, {}, =default, {})
- TEST_TEMPLATE( {}, {}, {}, =default)
- TEST_TEMPLATE( {}, {}, {}, {})
+ TEST_TEMPLATE(=default, =default, =default, )
+ TEST_TEMPLATE(=default, =default, , =default)
+ TEST_TEMPLATE(=default, =default, , )
+ TEST_TEMPLATE(=default, , =default, =default)
+ TEST_TEMPLATE(=default, , =default, )
+ TEST_TEMPLATE(=default, , , =default)
+ TEST_TEMPLATE(=default, , , )
+ TEST_TEMPLATE( , =default, =default, =default)
+ TEST_TEMPLATE( , =default, =default, )
+ TEST_TEMPLATE( , =default, , =default)
+ TEST_TEMPLATE( , =default, , )
+ TEST_TEMPLATE( , , =default, =default)
+ TEST_TEMPLATE( , , =default, )
+ TEST_TEMPLATE( , , , =default)
+ TEST_TEMPLATE( , , , )
#undef TEST_TEMPLATE
static_assert(is_trivially_copy_constructible_v<variant<DefaultNoexcept,
int, char, float, double>>, "");
Index: testsuite/23_containers/map/modifiers/try_emplace/1.cc
===================================================================
--- testsuite/23_containers/map/modifiers/try_emplace/1.cc (revision
254442)
+++ testsuite/23_containers/map/modifiers/try_emplace/1.cc (working copy)
@@ -39,6 +39,7 @@
{
val = other.val;
other.moved_from_assign = true;
+ return *this;
}
};
Index: testsuite/23_containers/unordered_map/modifiers/try_emplace.cc
===================================================================
--- testsuite/23_containers/unordered_map/modifiers/try_emplace.cc
(revision 254442)
+++ testsuite/23_containers/unordered_map/modifiers/try_emplace.cc
(working copy)
@@ -39,6 +39,7 @@
{
val = other.val;
other.moved_from_assign = true;
+ return *this;
}
};