Re: [PATCH] combine: Do not allow asm as I2 in a special case

[Jeff Law]

On 12/10/14 13:12, Segher Boessenkool wrote:

My rs6000 patch putting a clobber of the carry in every asm regressed
guality/pr41353-1.c.  This is because the asm (in f3 in that testcase,
for example) now is a PARALLEL, and the special case for I2 a parallel
and I3 a register move now triggers.  Before, when the asm was not a
parallel, can_combine_p disallowed combining I2.  The effect of the
change is that some debug info becomes invalid and is deleted later,
causing the testsuite regression.

Let's not allow combining an asm in this special case either.

Bootstrapped and tested on powerpc64-linux; okay for mainline?


Segher


2014-12-10  Segher Boessenkool  <seg...@kernel.crashing.org>

gcc/
        * combine.c (try_combine): Do not allow combining a PARALLEL I2
        with a register move I3 if that I2 is an asm.

OK.
jeff