On Thu, 1 Nov 2018 at 11:37, Jakub Jelinek <ja...@redhat.com> wrote:
>
> Hi!
>
> These two tests FAIL on i686-linux, because of extra warnings:
> /home/jakub/src/gcc/gcc/testsuite/gdc.dg/compilable.d:441:6: warning: MMX 
> vector return without MMX enabled changes the ABI [-Wpsabi]
> /home/jakub/src/gcc/gcc/testsuite/gdc.dg/simd.d:1138:8: warning: SSE vector 
> return without SSE enabled changes the ABI [-Wpsabi]
> /home/jakub/src/gcc/gcc/testsuite/gdc.dg/simd.d:1138:8: warning: SSE vector 
> argument without SSE enabled changes the ABI [-Wpsabi]
> The standard way to get around this is by adding -Wno-psabi option
> that disables these warnings.
>
> The following patch makes -Wno-psabi recognized by the D compiler and
> changes the two tests.
>
> Tested on i686-linux, ok for trunk?
>
> Note, other parts of that PR are not fixed.
>
> 2018-11-01  Jakub Jelinek  <ja...@redhat.com>
>
>         PR d/87824
>         * lang.opt (Wpsabi): New option.
>
>         * gdc.dg/simd.d: Add -Wno-psabi.
>         * gdc.dg/compilable.d: Likewise.
>
> --- gcc/d/lang.opt.jj   2018-11-01 07:43:33.760147379 +0100
> +++ gcc/d/lang.opt      2018-11-01 11:30:20.478011972 +0100
> @@ -130,6 +130,10 @@ Werror
>  D
>  ; Documented in common.opt
>
> +Wpsabi
> +D
> +; Documented in C
> +
>  Wspeculative
>  D
>  Warn from speculative compiles such as __traits(compiles).
> --- gcc/testsuite/gdc.dg/simd.d.jj      2018-11-01 07:43:34.568134010 +0100
> +++ gcc/testsuite/gdc.dg/simd.d 2018-11-01 11:25:14.894038430 +0100
> @@ -1,3 +1,4 @@
> +// { dg-options "-Wno-psabi" }
>  // { dg-do run { target hw } }
>  import core.simd;
>  import core.stdc.string;
> --- gcc/testsuite/gdc.dg/compilable.d.jj        2018-11-01 07:43:34.562134109 
> +0100
> +++ gcc/testsuite/gdc.dg/compilable.d   2018-11-01 11:25:01.712255255 +0100
> @@ -1,4 +1,4 @@
> -// { dg-options "-I $srcdir/gdc.dg -I $srcdir/gdc.dg/imports" }
> +// { dg-options "-I $srcdir/gdc.dg -I $srcdir/gdc.dg/imports -Wno-psabi" }
>  // { dg-additional-sources "imports/gdc27.d imports/gdc231.d" }
>  // { dg-do compile }
>
>

OK. Thanks.

-- 
Iain

Reply via email to