On Thu, Jan 29, 2015 at 03:44:25PM +0100, Uros Bizjak wrote:
> Hello!
>
> > So here's the updated patch which handles all 4 testcases from the PR as
> > well as a couple of my own.
>
> @@ -0,0 +1,33 @@
> +/* PR 15184 first two tests, plus two addition ones. */
> +/* { dg-do compile } */
> +/* { dg-options "-O2 -m32 -march=pentiumpro" } */
>
> No, we don't want -m32 in dg-options. Please write this part as:
>
> /* { dg-do compile { target ia32 } } */
> /* { dg-options "-O2 -march=pentiumpro" } */
Or
/* { dg-do compile } */
/* { dg-options "-O2" } */
/* { dg-additional-options "-march=pentiumpro" { target ia32 } } */
if there is nothing ia32 specific on the testcase, just that you want
to use that -march option if you are compiling it as 32-bit.
Jakub
