Re: C++ PATCH to remove WITH_CLEANUP_EXPR handling

2017-07-03 Thread Jason Merrill
Absolutely. On Mon, Jul 3, 2017 at 5:35 AM, Marek Polacek wrote: > On Thu, Jun 29, 2017 at 05:44:25PM -0400, Jason Merrill wrote: >> The C++ front end hasn't generated WITH_CLEANUP_EXPR in a very long >> time (20+ years?), so there's no need to handle it. > > Heh. Found another one; is this patc

Re: C++ PATCH to remove WITH_CLEANUP_EXPR handling

2017-07-03 Thread Marek Polacek
On Thu, Jun 29, 2017 at 05:44:25PM -0400, Jason Merrill wrote: > The C++ front end hasn't generated WITH_CLEANUP_EXPR in a very long > time (20+ years?), so there's no need to handle it. Heh. Found another one; is this patch ok if it passes testing? 2017-07-03 Marek Polacek * c-warn.

Re: C++ PATCH to remove WITH_CLEANUP_EXPR handling

2017-06-30 Thread Nathan Sidwell
On 06/29/2017 05:44 PM, Jason Merrill wrote: The C++ front end hasn't generated WITH_CLEANUP_EXPR in a very long time (20+ years?), so there's no need to handle it. I see dead code nathan -- Nathan Sidwell

C++ PATCH to remove WITH_CLEANUP_EXPR handling

2017-06-29 Thread Jason Merrill
The C++ front end hasn't generated WITH_CLEANUP_EXPR in a very long time (20+ years?), so there's no need to handle it. Tested x86_64-pc-linux-gnu, applying to trunk. commit 8ce76bba3bf7866a2338125b4c55b0a1e3ce9221 Author: Jason Merrill Date: Thu Jun 29 15:56:20 2017 -0400 * conste