Bug 5200 - Store Motion is broken
Summary: Store Motion is broken
Status: RESOLVED FIXED
Alias: None
Product: gcc
Classification: Unclassified
Component: rtl-optimization (show other bugs)
Version: unknown
: P3 normal
Target Milestone: 3.4.0
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2001-12-26 20:06 UTC by Andreas Jaeger
Modified: 2003-06-12 00:15 UTC (History)
6 users (show)

See Also:
Host:
Target:
Build:
Known to work:
Known to fail:
Last reconfirmed:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Jaeger 2001-12-26 20:06:00 UTC
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
Comment 1 Andreas Jaeger 2001-12-26 20:06:00 UTC
Fix:
http://gcc.gnu.org/ml/gcc-patches/2002-03/msg00474.html
Comment 2 Craig Rodrigues 2002-03-10 12:59:29 UTC
State-Changed-From-To: open->closed
State-Changed-Why: http://gcc.gnu.org/ml/gcc-patches/2002-03/msg00474.html
Comment 3 Andreas Jaeger 2002-03-10 22:05:54 UTC
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

Comment 4 Craig Rodrigues 2002-03-18 13:41:47 UTC
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?
Comment 5 Jakub Jelinek 2002-03-18 17:03:31 UTC
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

Comment 6 Nathanael C. Nerode 2003-01-01 16:25:39 UTC
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?

Comment 7 s.bosscher 2003-05-12 13:37:12 UTC
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
 

Comment 8 Andreas Jaeger 2003-05-12 13:58:57 UTC
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
Comment 9 Steven Bosscher 2003-05-24 16:56:56 UTC
Zdenek, could it be that this PR was fixed with your recent rewrite of store motion?

Thanks,
Steven
Comment 10 Zdenek Dvorak 2003-06-04 08:17:57 UTC
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
Comment 11 Andreas Jaeger 2003-06-09 05:55:54 UTC
This should be fixed with Zdenek's changes.