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