On 14/07/15 13:55, Segher Boessenkool wrote:
On Tue, Jul 14, 2015 at 09:18:06AM +0100, Kyrill Tkachov wrote:
Before combine, you have two insns, a negation and an abs, so that is
not so very strange :-)
Oh, hrm, my aarch64 cross was three months old, and this now changed.
Or I messed up.  Sorry for the noise.

It does look like the if_then_else stuff in combine (and simplify-rtx)
could use some improvement, probably more than just fixing this one
case.  Or look at the first huge case in combine_simplify_rtx, that
looks eerily similar.

Yeah, removing an early return in combine_simplify_rtx and
adding the simplification to simplify_unary_operation did
the trick.

I'll see if it survives testing.

Thanks,
Kyrill



Segher


Reply via email to