Fix failing tests after PR libstdc++/85965
diff mbox series

Message ID 176edde6-81d4-9685-05db-8568803f7e3d@gmail.com
State New
Headers show
Series
  • Fix failing tests after PR libstdc++/85965
Related show

Commit Message

François Dumont July 18, 2019, 5:41 a.m. UTC
Since commit 5d3695d03b7bdade9f4d05d2b those tests are failing.

     * testsuite/23_containers/unordered_map/48101_neg.cc: Adapt dg-error
     after PR libstdc++/85965 fix.
     * testsuite/23_containers/unordered_multimap/48101_neg.cc: Likewise.
     * testsuite/23_containers/unordered_multiset/48101_neg.cc: Likewise.
     * testsuite/23_containers/unordered_set/48101_neg.cc

It is quite trivial but I wonder if there is another plan to restore 
those static assertions differently.

Ok to commit ?

François

Comments

Jonathan Wakely July 18, 2019, 10:18 a.m. UTC | #1
On 18/07/19 07:41 +0200, François Dumont wrote:
>Since commit 5d3695d03b7bdade9f4d05d2b those tests are failing.
>
>    * testsuite/23_containers/unordered_map/48101_neg.cc: Adapt dg-error
>    after PR libstdc++/85965 fix.
>    * testsuite/23_containers/unordered_multimap/48101_neg.cc: Likewise.
>    * testsuite/23_containers/unordered_multiset/48101_neg.cc: Likewise.
>    * testsuite/23_containers/unordered_set/48101_neg.cc
>
>It is quite trivial but I wonder if there is another plan to restore 
>those static assertions differently.
>
>Ok to commit ?

No. I don't see these failures. With the first change applied, I see a
new failure.

The patch seems wrong.
François Dumont July 18, 2019, 9:11 p.m. UTC | #2
Got it, it is my PR 68303 patch which was introducing this regression. I 
fix it to restore those assertions.

You'll see once the awaiting hashtable patches are in...

On 7/18/19 12:18 PM, Jonathan Wakely wrote:
> On 18/07/19 07:41 +0200, François Dumont wrote:
>> Since commit 5d3695d03b7bdade9f4d05d2b those tests are failing.
>>
>>     * testsuite/23_containers/unordered_map/48101_neg.cc: Adapt dg-error
>>     after PR libstdc++/85965 fix.
>>     * testsuite/23_containers/unordered_multimap/48101_neg.cc: Likewise.
>>     * testsuite/23_containers/unordered_multiset/48101_neg.cc: Likewise.
>>     * testsuite/23_containers/unordered_set/48101_neg.cc
>>
>> It is quite trivial but I wonder if there is another plan to restore 
>> those static assertions differently.
>>
>> Ok to commit ?
>
> No. I don't see these failures. With the first change applied, I see a
> new failure.
>
> The patch seems wrong.
>
>
>

Patch
diff mbox series

diff --git a/libstdc++-v3/testsuite/23_containers/unordered_map/48101_neg.cc b/libstdc++-v3/testsuite/23_containers/unordered_map/48101_neg.cc
index 8d823dfa476..77c0e9ce681 100644
--- a/libstdc++-v3/testsuite/23_containers/unordered_map/48101_neg.cc
+++ b/libstdc++-v3/testsuite/23_containers/unordered_map/48101_neg.cc
@@ -27,7 +27,4 @@  test01()
   c2.find(2); // { dg-error "here" }
 }
 
-// { dg-error "hash function must be invocable" "" { target *-*-* } 0 }
-// { dg-error "key equality predicate must be invocable" "" { target *-*-* } 0 }
-// { dg-prune-output "use of deleted function" }
 // { dg-prune-output "no match for call" }
diff --git a/libstdc++-v3/testsuite/23_containers/unordered_multimap/48101_neg.cc b/libstdc++-v3/testsuite/23_containers/unordered_multimap/48101_neg.cc
index a81615b3607..7db7dcb2b5d 100644
--- a/libstdc++-v3/testsuite/23_containers/unordered_multimap/48101_neg.cc
+++ b/libstdc++-v3/testsuite/23_containers/unordered_multimap/48101_neg.cc
@@ -27,7 +27,4 @@  test01()
   c2.find(2); // { dg-error "here" }
 }
 
-// { dg-error "hash function must be invocable" "" { target *-*-* } 0 }
-// { dg-error "key equality predicate must be invocable" "" { target *-*-* } 0 }
-// { dg-prune-output "use of deleted function" }
 // { dg-prune-output "no match for call" }
diff --git a/libstdc++-v3/testsuite/23_containers/unordered_multiset/48101_neg.cc b/libstdc++-v3/testsuite/23_containers/unordered_multiset/48101_neg.cc
index 03ddb898d6c..444ca210c58 100644
--- a/libstdc++-v3/testsuite/23_containers/unordered_multiset/48101_neg.cc
+++ b/libstdc++-v3/testsuite/23_containers/unordered_multiset/48101_neg.cc
@@ -29,8 +29,5 @@  test01()
 }
 
 // { dg-error "non-const, non-volatile value_type" "" { target *-*-* } 0 }
-// { dg-error "hash function must be invocable" "" { target *-*-* } 0 }
-// { dg-error "key equality predicate must be invocable" "" { target *-*-* } 0 }
-// { dg-prune-output "use of deleted function" }
 // { dg-prune-output "must have the same value_type as its allocator" }
 // { dg-prune-output "no match for call" }
diff --git a/libstdc++-v3/testsuite/23_containers/unordered_set/48101_neg.cc b/libstdc++-v3/testsuite/23_containers/unordered_set/48101_neg.cc
index e79d3769248..2c02620bccb 100644
--- a/libstdc++-v3/testsuite/23_containers/unordered_set/48101_neg.cc
+++ b/libstdc++-v3/testsuite/23_containers/unordered_set/48101_neg.cc
@@ -29,8 +29,5 @@  test01()
 }
 
 // { dg-error "non-const, non-volatile value_type" "" { target *-*-* } 0 }
-// { dg-error "hash function must be invocable" "" { target *-*-* } 0 }
-// { dg-error "key equality predicate must be invocable" "" { target *-*-* } 0 }
-// { dg-prune-output "use of deleted function" }
 // { dg-prune-output "must have the same value_type as its allocator" }
 // { dg-prune-output "no match for call" }