On Mon, 24 Sep 2018, Alexandre Oliva wrote:
> On Sep 18, 2018, Richard Biener <rguent...@suse.de> wrote:
>
> >> +# ifdef __SSE__
> >> +# define T __int128
>
> Rainer reported this didn't work on 32-bit x86 with SSE enabled.
> Here's a patch that fixes it. Ok to install? Tested on x86_64,
> including such combinations as -m32, -m32/-msse, and -m32/-mno-sse.
Works for me.
>
> [PR87054] adjust testcase for 32-bit x86
>
> From: Alexandre Oliva <ol...@adacore.com>
>
> The test assumed __int128 to be available whenever __SSE__ was
> defined, but this assumption doesn't hold on 32-bit x86. Fixed.
>
> for gcc/testsuite/ChangeLog
>
> PR middle-end/87054
> * gcc.dg/pr87054.c: Adjust for no __int128 on x86.
> ---
> gcc/testsuite/gcc.dg/pr87054.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/testsuite/gcc.dg/pr87054.c b/gcc/testsuite/gcc.dg/pr87054.c
> index 4ca2b62d2c7c9..3cb3b7dac6586 100644
> --- a/gcc/testsuite/gcc.dg/pr87054.c
> +++ b/gcc/testsuite/gcc.dg/pr87054.c
> @@ -2,7 +2,7 @@
> // { dg-options "-O2" }
>
> #ifndef T
> -# ifdef __SSE__
> +# if __SIZEOF_INT128__ && defined __SSE__
> # define T __int128
> # else
> # define T long
>
