[PATCH] Adjust hoist-register-pressure* testcases to work for S/390

Jeff Law law@redhat.com
Thu Apr 24 22:27:00 GMT 2014


On 04/10/14 02:00, Andreas Krebbel wrote:
> Hi,
>
> the hoist-register-pressure testcases currently fail on S/390 since
> the rtl hoist pass requires that the expression to be hoisted can be
> assigned without clobbering cc.  We do not have a 32 bit add which
> does not clobber cc.
>
> On 64 bit we might use load address if the operands are DImode.  On 31
> bit we are out of luck.
>
> The attached patch turns the int types into long for these testcases
> and disables them for s390 31bit.
>
> I've verified that they still succeed on power and x86_64.
>
> Ok?
>
> Bye,
>
> -Andreas-
>
>
> 2014-04-10  Andreas Krebbel  <Andreas.Krebbel@de.ibm.com>
>
> 	* gcc.dg/hoist-register-pressure-1.c: Replace int with long.
> 	Disable asm scan for s390.
> 	* gcc.dg/hoist-register-pressure-2.c: Likewise.
> 	* gcc.dg/hoist-register-pressure-3.c: Likewise.
OK for the trunk.

Thanks,
Jeff



More information about the Gcc-patches mailing list