On 10/22/19 2:13 AM, Claudiu Zissulescu wrote:
> From: Shahab Vahedi <sha...@synopsys.com>
> 
> Hi Andrew,
> 
> The movsi_ne variants are in a wrong order, leading to wrong
> computation of the internal attribute "cond". Hence, to errors when
> outputting annul-true or annul-false instructions. Testcase added.
> 
> The patch needs to go for trunk and gcc9 branch.
> 
> OK to apply?
> Claudiu
> 
> gcc/
> xxxx-xx-xx  Claudiu Zissulescu  <claz...@synopsys.com>
>           Shahab Vahedi  <sha...@synopsys.com>
> 
>       * config/arc/arc.md (movsi_ne): Reorder instruction variants.
> 
> testsuite/
> xxxx-xx-xx  Shahab Vahedi  <sha...@synopsys.com>
> 
>       * gcc.target/arc/delay-slot-limm.c: New test.
So I'm going to have to trust you on this one.  It looks like you did
more than just reorder the alternatives.  For example, the constraints
for operand0 look significantly different to me.  THey're slightly
different for operand1 as well (LR rather than Lc).

Anyway, OK for the trunk.

jeff

Reply via email to