This is the mail archive of the gcc-bugs@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395



------- 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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]