Store Motion is currently broken. For details look into these messages and the discussion around it: http://gcc.gnu.org/ml/gcc-patches/2001-11/msg00777.html http://gcc.gnu.org/ml/gcc-patches/2001-11/msg00751.html http://gcc.gnu.org/ml/gcc-patches/2001-08/msg00268.html PR 5172 might also be a store motion bug. Release: unknown Environment: Any environment
Fix: http://gcc.gnu.org/ml/gcc-patches/2002-03/msg00474.html
State-Changed-From-To: open->closed State-Changed-Why: http://gcc.gnu.org/ml/gcc-patches/2002-03/msg00474.html
From: Andreas Jaeger <aj@suse.de> To: rodrigc@gcc.gnu.org Cc: Subject: Re: optimization/5200: Store Motion is broken Date: Sun, 10 Mar 2002 22:05:54 +0100 Hi, I don't agree - this solves the problem on the branch but what about mainline? The issue is solved for 3.1 but open for 3.2, Andreas -- Andreas Jaeger SuSE Labs aj@suse.de private aj@arthur.inka.de http://www.suse.de/~aj
State-Changed-From-To: closed->feedback State-Changed-Why: Jakub, according to Andreas, this problem is fixed on the gcc 3.1 branch by your patch, but not on the mainline. Care to comment?
From: Jakub Jelinek <jakub@redhat.com> To: rodrigc@gcc.gnu.org, aj@suse.de, gcc-bugs@gcc.gnu.org, gcc-prs@gcc.gnu.org, nobody@gcc.gnu.org, gcc-gnats@gcc.gnu.org Cc: Subject: Re: optimization/5200: Store Motion is broken Date: Mon, 18 Mar 2002 17:03:31 -0500 On Mon, Mar 18, 2002 at 09:41:50PM -0000, rodrigc@gcc.gnu.org wrote: > Jakub, according to Andreas, this problem is > fixed on the gcc 3.1 branch by your patch, but not on the mainline. > > Care to comment? On mainline IMHO it should be fixed the hard way, ie. apply the patch to fix store motion checks and then work out how to fix all store motion bugs. This is not possible for the branch, since when sm basically never worked in 3.1 development cycle, it would be actually a completely new feature coming in hard feature freeze. Jakub
From: Nathanael Nerode <neroden@twcny.rr.com> To: gcc-gnats@gcc.gnu.org, aj@suse.de, gcc-bugs@gcc.gnu.org, nobody@gcc.gnu.org, jakub@redhat.com, rodrigc@gcc.gnu.org Cc: Subject: Re: optimization/5200: Store Motion is broken Date: Wed, 1 Jan 2003 16:25:39 -0500 What's the status of this on mainline? Did it get fixed?
From: Steven Bosscher <s.bosscher@student.tudelft.nl> To: gcc-prs@gcc.gnu.org, aj@suse.de, gcc-bugs@gcc.gnu.org, gcc-gnats@gcc.gnu.org, jakub@redhat.com, rodrigc@gcc.gnu.org, nobody@gcc.gnu.org Cc: Subject: Re: optimization/5200: Store Motion is broken Date: Mon, 12 May 2003 13:37:12 +0200 Hi, Nathaneal Nerode asked for confirmation of this PR in early January, but there was no reply. Usually that would mean "close it, no feedback for more than three months". But this is special because so many people have looked at it and because Zdenek was working on a store motion rewrite some time ago. Can somebody look at it and see if the problem still exists, and change the status of the PR accordingly (ie. analyzed or closed)? Thanks, Steven http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=5200
From: Andreas Jaeger <aj@suse.de> To: Steven Bosscher <s.bosscher@student.tudelft.nl> Cc: gcc-prs@gcc.gnu.org, gcc-bugs@gcc.gnu.org, gcc-gnats@gcc.gnu.org, jakub@redhat.com, rodrigc@gcc.gnu.org, nobody@gcc.gnu.org, Zdenek Dvorak <rakdver@atrey.karlin.mff.cuni.cz> Subject: Re: optimization/5200: Store Motion is broken Date: Mon, 12 May 2003 13:58:57 +0200 Steven Bosscher <s.bosscher@student.tudelft.nl> writes: > Hi, > > Nathaneal Nerode asked for confirmation of this PR in early January, > but there was no reply. > > Usually that would mean "close it, no feedback for more than three > months". But this is special because so many people have looked at > it and because Zdenek was working on a store motion rewrite some > time ago. > > Can somebody look at it and see if the problem still exists, and change > the status of the PR accordingly (ie. analyzed or closed)? Zdenek is on vacation right now and will be back next month. Please remind him again, when he's back. > > http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=5200 Thanks for taking care of this, I agree this is the right way for this bug, Andreas -- Andreas Jaeger SuSE Labs aj@suse.de private aj@arthur.inka.de http://www.suse.de/~aj
Zdenek, could it be that this PR was fixed with your recent rewrite of store motion? Thanks, Steven
Subject: Re: Store Motion is broken Hello, > Zdenek, could it be that this PR was fixed with your recent rewrite of store motion? I am not entirely sure (mainly because the PR is quite fuzzily formulated -- "Store Motion is broken, here is a bunch of unrelated issues why"). All the issues I was able to get to from the PR are already resolved either by my patch or some previous ones. Zdenek
This should be fixed with Zdenek's changes.