Re: optimization/5200: Store Motion is broken

Re: optimization/5200: Store Motion is broken

[Jakub Jelinek]

The following reply was made to PR optimization/5200; it has been noted by GNATS.

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

Re: optimization/5200: Store Motion is broken

[Andreas Jaeger]

The following reply was made to PR optimization/5200; it has been noted by GNATS.

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

Re: optimization/5200: Store Motion is broken

[Steven Bosscher]

The following reply was made to PR optimization/5200; it has been noted by GNATS.

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
 

Re: optimization/5200: Store Motion is broken

[Nathanael Nerode]

The following reply was made to PR optimization/5200; it has been noted by GNATS.

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?

Re: optimization/5200: Store Motion is broken

[Andreas Jaeger]

The following reply was made to PR optimization/5200; it has been noted by GNATS.

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

Re: optimization/5200: Store Motion is broken

[rodrigc]

Synopsis: Store Motion is broken

State-Changed-From-To: closed->feedback
State-Changed-By: rodrigc
State-Changed-When: Mon Mar 18 13:41:47 2002
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?

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=5200

Re: optimization/5200: Store Motion is broken

[rodrigc]

Synopsis: Store Motion is broken

State-Changed-From-To: open->closed
State-Changed-By: rodrigc
State-Changed-When: Sun Mar 10 12:59:29 2002
State-Changed-Why:
    http://gcc.gnu.org/ml/gcc-patches/2002-03/msg00474.html

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=5200

optimization/5200: Store Motion is broken

[aj]

>Number:         5200
>Category:       optimization
>Synopsis:       Store Motion is broken
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    unassigned
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed Dec 26 20:06:00 PST 2001
>Closed-Date:
>Last-Modified:
>Originator:     Andreas Jaeger
>Release:        unknown-1.0
>Organization:
>Environment:
Any environment
>Description:
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.
>How-To-Repeat:

>Fix:
Disable Store motion as in Daniel's Patch from August
is one work-around.
>Release-Note:
>Audit-Trail:
>Unformatted: