SH: cmpeqdi splitter messes up the CFG

Alexandre Oliva aoliva@redhat.com
Wed Aug 30 10:37:00 GMT 2000


On Aug 29, 2000, Jeffrey A Law <law@cygnus.com> wrote:

>> * config/sh/sh.md (cmpeqsi_ior_t, cmpeqsi_and_t): New insns.
>> (cmpeqdi_t splitter): Use cmpeqsi_and_t instead of emitting jumps
>> and labels.

>> +	bt 0f\;tst	%0,%0\;0:
>> +	bt 0f\;cmp/eq	%1,%0\;0:
>> +	bt 0f\;cmp/eq	%1,%0\;0:"
>> +  [(set_attr "length" "4")])

> I'd recommend against using gas specific extensions like the forward/backward
> labels.

Ok.  How about `.Leqsi%='?

> I don't see where/how the cmpeqsi_ior_t pattern is used.

It isn't, AFAIK.  I put it in just for completeness.  Then, before
throwing it away as useless, I thought combine and ifcvt could benefit
from it, so left it in.

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                  aoliva@{cygnus.com, redhat.com}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist    *Please* write to mailing lists, not to me



More information about the Gcc-patches mailing list