On Fri, Jul 27, 2012 at 8:58 AM, Segher Boessenkool <seg...@kernel.crashing.org> wrote:
> I changed the test to INT_REGNO_P || FP_REGNO_P; this isn't > totally obvious since INT_REGNO_P includes ap and fp, but those > are covered by the other arms of the conditional already: in > fact, it probably would be better to rewrite the whole thing > to simply disallow LR, CTR, CA (and MQ ;-) ); this would > express the purpose much better. But that's not for this > patch. Regtested again, all three patches committed. Please undo that change. You made the test for that heavily used function even more expensive. Thanks, David
