Re: RFA: Combine issue

2005-08-04 Thread Eric Botcazou
> I think I understand what was being attempted now. IIUC, the logic > should have been (again, for the QI/SI instance): > >If at least _26_ bits are set, and if c1 is < 64, ok to make the > transformation. Yes, that was the intended logic. > In this case, I don't see an instance where "comp

Re: RFA: Combine issue

2005-08-04 Thread Josh Conner
Am I misinterpreting the logic? Am I missing something fundamental? I appreciate any feedback / pointers / clues / etc... Nothing like hitting the send button to make the lightbulb go on. I think I understand what was being attempted now. IIUC, the logic should have been (again, for the

RFA: Combine issue

2005-08-04 Thread Josh Conner
I'm seeing invalid code produced for a simple test case targeting arm- none-elf (attached), which I believe is caused by an invalid transformation in simplify_comparison. It's transforming code of the form: (compare (subreg:QI (plus (reg:SI) (-1))) (-3)) into: (compare (plu