On Wed, Apr 04, 2018 at 12:14:43PM +0200, Martin Liška wrote:
> Hi.
>
> It's test-case workaround, tested on x86_64 and powerpc with both -m64 and
> -m32.
>
> Ready for trunk?
> Thanks,
> Martin
>
> gcc/testsuite/ChangeLog:
>
> 2018-04-04 Martin Liska <mli...@suse.cz>
>
> PR sanirizer/85174
> * c-c++-common/asan/pointer-compare-1.c: Disable section anchors
> and msdata as a workaround for powerpc.
> ---
> gcc/testsuite/c-c++-common/asan/pointer-compare-1.c | 3 +++
> 1 file changed, 3 insertions(+)
>
>
> diff --git a/gcc/testsuite/c-c++-common/asan/pointer-compare-1.c
> b/gcc/testsuite/c-c++-common/asan/pointer-compare-1.c
> index cf67fe98bee..d4d7784785c 100644
> --- a/gcc/testsuite/c-c++-common/asan/pointer-compare-1.c
> +++ b/gcc/testsuite/c-c++-common/asan/pointer-compare-1.c
> @@ -2,6 +2,9 @@
> /* { dg-set-target-env-var ASAN_OPTIONS
> "detect_invalid_pointer_pairs=2:halt_on_error=0" } */
> /* { dg-options "-fsanitize=address,pointer-compare" } */
>
> +/* TODO: remove me after PR sanitizer/82501 is resolved. */
> +/* { dg-additional-options "-fno-section-anchors -msdata=none" { target {
> powerpc*-*-* } } } */
I'd suggest
/* { dg-additional-options "-fno-section-anchors" } */
and only -msdata=none for powerpc* only. We have several other section
anchors targets, you can perhaps check gcc-testresults how the test looks
like there. And use FIXME instead of TODO.
Ok with that change.
Jakub
