Re: [PATCH, rs6000] pr65479 Add -fasynchronous-unwind-tables when the -fsanitize=address option is seen.

[Bill Seurer <seurer at linux dot vnet dot ibm dot com>]

On 01/06/2017 01:48 AM, Segher Boessenkool wrote:
> On Thu, Jan 05, 2017 at 01:21:40PM -0600, Bill Seurer wrote:
> > (backport from trunk to gcc 6)
> >
> > This patch adds the -fasynchronous-unwind-tables option to compilations when
> > the -fsanitize=address option is seen but not if any
> > -fasynchronous-unwind-tables options were already specified.
> > -fasynchronous-unwind-tables causes a full strack trace to be produced when
> > the sanitizer detects an error.  Without the full trace several of the asan
> > test cases fail on powerpc.
> >
> > See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65479 for more information.
> >
> > Bootstrapped and tested on powerpc64le-unknown-linux-gnu,
> > powerpc64be-unknown-linux-gnu, and x86_64-pc-linux-gnu with no regressions.
> > Is this ok for trunk?
>
> It already is on trunk...  You mean the branches :-)  It is okay for all
> open release branches (i.e., 5 and 6).  Or does it not apply to GCC 5?
>
> Thanks,
>
>
> Segher

Yes, I meant the 6 branch.  I only tried it on 6 but it should be fine 
for 5 too.


> > 2017-01-05  Bill Seurer  <seurer@linux.vnet.ibm.com>
> >
> > 	PR sanitizer/65479
> > 	* config/rs6000/rs6000.c (rs6000_option_override_internal): Add
> > 	-fasynchronous-unwind-tables option when -fsanitize=address is
> > 	specified.


--

-Bill Seurer