------- Comment #11 from bonzini at gnu dot org 2009-06-23 13:09 ------- > Yes, but that only means that the operands can be swapped *if* > swap_commutative_operands_p() returns true. Due to the funny precedence that > does not happen. > > Matching procedures do not take commutativeness into account.
That would mean that PR28690 screwed up two-address machines big. In that case my opinion of the patch would change. Still, I don't think a target hook is the solution. Even if it adds hack over hack, having the "funny precedence" rules only before reload could be a solution. I'm pretty sure that we are talking about two different things. :-) -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37053
