This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] pr65479 Add -fasynchronous-unwind-tables when the -fsanitize=address option is seen.
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Bill Seurer <seurer at linux dot vnet dot ibm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 6 Jan 2017 01:48:16 -0600
- Subject: Re: [PATCH, rs6000] pr65479 Add -fasynchronous-unwind-tables when the -fsanitize=address option is seen.
- Authentication-results: sourceware.org; auth=none
- References: <a3456b9e-4ab4-ca6e-5403-5cd1f132370b@linux.vnet.ibm.com>
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
> 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.