Re: [PATCH] c++: Add testcase for now fixed issue [PR117324]

2025-02-16 Thread Simon Martin
On 14 Feb 2025, at 17:51, Marek Polacek wrote: > On Fri, Feb 14, 2025 at 04:45:01PM +, Simon Martin wrote: >> Hi Marek, >> >> On 14 Feb 2025, at 17:27, Marek Polacek wrote: >> >>> On Fri, Feb 14, 2025 at 04:20:07PM +, Simon Martin wrote: The case in this PR does not ICE anymore after

Re: [PATCH] c++: Add testcase for now fixed issue [PR117324]

2025-02-14 Thread Marek Polacek
; This fails with C++14: > > FAIL: g++.dg/parse/defarg19.C -std=c++14 (test for excess errors) > Doh, indeed :-/ > > > I think you want c++14_down. > Thanks, fixed in the updated attached patch, tested with >make -C gcc check-c++-all RUNTESTFLAGS="dg.exp=defarg19.C&quo

Re: [PATCH] c++: Add testcase for now fixed issue [PR117324]

2025-02-14 Thread Simon Martin
++11_down } } > > This fails with C++14: > FAIL: g++.dg/parse/defarg19.C -std=c++14 (test for excess errors) Doh, indeed :-/ > I think you want c++14_down. Thanks, fixed in the updated attached patch, tested with make -C gcc check-c++-all RUNTESTFLAGS="dg.exp=defarg19.C"

Re: [PATCH] c++: Add testcase for now fixed issue [PR117324]

2025-02-14 Thread Marek Polacek
On Fri, Feb 14, 2025 at 04:20:07PM +, Simon Martin wrote: > The case in this PR does not ICE anymore after the fix for PR118319. > > This patch simply adds the case to the testsuite. > > Successfully tested on x86_64-apple-darwin19.6.0. > > PR c++/117324 > > gcc/testsuite/ChangeLog: >

[PATCH] c++: Add testcase for now fixed issue [PR117324]

2025-02-14 Thread Simon Martin
The case in this PR does not ICE anymore after the fix for PR118319. This patch simply adds the case to the testsuite. Successfully tested on x86_64-apple-darwin19.6.0. PR c++/117324 gcc/testsuite/ChangeLog: * g++.dg/parse/defarg19.C: New test. --- gcc/testsuite/g++.dg/parse/