On 02/21/2017 07:48 AM, Segher Boessenkool wrote:
REGNO can only be called on REGs, not SUBREGs; and INTVAL does not work
on REGs.

2017-02-21  Segher Boessenkool  <seg...@kernel.crashing.org>

        * config/microblaze/microblaze.c (microblaze_expand_shift): Do not
        test for register moves to themselves.
        * config/microblaze/microblaze.md (*ashlsi3_byone, *ashrsi3_byone,
        *lshrsi3_byone): Test for const1_rtx instead of calling INTVAL on
        something that isn't necessarily a CONST_INT.
So I wanted to make sure that the avoidance of a nop-move resulting from a nop-shift wasn't because the ISA couldn't encode a nop-move (and yes, I've run into such ISAs).

The microblaze doesn't seem to have any trouble encoding a nop move, so if we were to generate one due to this change it won't cause a problem.


OK for the trunk.

Jeff

Reply via email to