From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 19582 invoked by alias); 23 Jun 2009 13:09:44 -0000 Received: (qmail 19536 invoked by uid 48); 23 Jun 2009 13:09:32 -0000 Date: Tue, 23 Jun 2009 13:09:00 -0000 Message-ID: <20090623130932.19535.qmail@sourceware.org> X-Bugzilla-Reason: CC References: Subject: [Bug middle-end/37053] [4.3/4.4/4.5 regression] ICE in reload_cse_simplify_operands, at postreload.c:395 In-Reply-To: Reply-To: gcc-bugzilla@gcc.gnu.org To: gcc-bugs@gcc.gnu.org From: "bonzini at gnu dot org" Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org X-SW-Source: 2009-06/txt/msg01696.txt.bz2 ------- 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