From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7194 invoked by alias); 27 Jan 2015 03:21:21 -0000 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 Received: (qmail 7071 invoked by uid 48); 27 Jan 2015 03:21:08 -0000 From: "amker at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug tree-optimization/62173] [5.0 regression] 64bit Arch can't ivopt while 32bit Arch can Date: Tue, 27 Jan 2015 03:21:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: tree-optimization X-Bugzilla-Version: 5.0 X-Bugzilla-Keywords: missed-optimization X-Bugzilla-Severity: normal X-Bugzilla-Who: amker at gcc dot gnu.org X-Bugzilla-Status: ASSIGNED X-Bugzilla-Priority: P1 X-Bugzilla-Assigned-To: jiwang at gcc dot gnu.org X-Bugzilla-Target-Milestone: 5.0 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-SW-Source: 2015-01/txt/msg02989.txt.bz2 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62173 --- Comment #25 from amker at gcc dot gnu.org --- (In reply to Jiong Wang from comment #24) > (In reply to amker from comment #23) > > partially agree. > > at least for the single use case given by Seb, I think tree ivopt should do > it. (I verified clang do ivopt correctly for the case) > > for the rtl re-associate, it's a little bit painful from my experiment > experiences. as it's not always good to reassociate virtual_frame + offset, > we can only benefit if it's in loop, because the re-associate will increase > register pressure, there will be situations that more callee-saved regs > used, and finally we run into unncessary push/pop in pro/epilogue... and I > haven't found a good place where we can safely re-use existed rtl info and > do the rtl re-association as I am afraid rebuild those rtl info will cause > compile time penalty. Yes, The ivopt's issue that it doesn't treat "A[d]" as an address type iv use should be fixed anyway.