Re: PATCH to add -flifetime-dse=1

2016-02-24 Thread Jason Merrill
On 02/24/2016 10:25 AM, Jakub Jelinek wrote: That should be @option{-flifetime-dse=1} I think. Shouldn't -flifetime-dse= be also in @opindex at the beginning of the paragraph, and documented what the values mean (0 equivalent of -fno-lifetime-dse (or document it vice versa) and 2 full lifetime d

Re: PATCH to add -flifetime-dse=1

2016-02-24 Thread Jakub Jelinek
On Wed, Feb 24, 2016 at 10:15:35AM -0500, Jason Merrill wrote: > --- a/gcc/doc/invoke.texi > +++ b/gcc/doc/invoke.texi > @@ -6809,7 +6809,10 @@ value, and any changes during the lifetime of the > object are dead when > the object is destroyed. Normally dead store elimination will take > advanta

PATCH to add -flifetime-dse=1

2016-02-24 Thread Jason Merrill
Various projects have been having trouble with the new -flifetime-dse clobber at the beginning of the constructor, so this patch allows them to pass -flifetime-dse=1 to disable that clobber while still keeping the one at the end of the destructor. Tested x86_64-pc-linux-gnu, applying to trunk.