[Bug target/28966] New: -maltivec -m32 causes the stack to be saved and restored even though there is no need for it

[Bug target/28966] New: -maltivec -m32 causes the stack to be saved and restored even though there is no need for it

[pinskia at gcc dot gnu dot org]

Testcase:
int f(void)
{
  return 0;
}
This produces with -maltivec -O2:
f:
        stwu 1,-16(1)
        li 3,0
        addi 1,1,16
        blr

With just -O2, we get:
f:
        li 3,0
        blr


-- 
           Summary: -maltivec -m32 causes the stack to be saved and restored
                    even though there is no need for it
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Keywords: missed-optimization
          Severity: enhancement
          Priority: P3
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: pinskia at gcc dot gnu dot org
GCC target triplet: powerpc-linux-gnu


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28966

[Bug target/28966] -maltivec -m32 causes the stack to be saved and restored even though there is no need for it

[nathan at gcc dot gnu dot org]

------- Comment #1 from nathan at gcc dot gnu dot org  2006-12-18 18:36 -------
*** Bug 29248 has been marked as a duplicate of this bug. ***


-- 

nathan at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |atgraham at gmail dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28966

[Bug target/28966] -maltivec -m32 causes the stack to be saved and restored even though there is no need for it

[nathan at gcc dot gnu dot org]

------- Comment #2 from nathan at gcc dot gnu dot org  2006-12-18 18:37 -------
patch here. http://gcc.gnu.org/ml/gcc-patches/2006-12/msg01260.html


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28966

[Bug target/28966] -maltivec -m32 causes the stack to be saved and restored even though there is no need for it

[nathan at gcc dot gnu dot org]

-- 

nathan at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |nathan at gcc dot gnu dot
                   |dot org                     |org
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-12-21 09:03:35
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28966

[Bug target/28966] -maltivec -m32 causes the stack to be saved and restored even though there is no need for it

[nathan at gcc dot gnu dot org]

------- Comment #3 from nathan at gcc dot gnu dot org  2006-12-21 09:04 -------
Fixed on mainline
2006-12-21  Nathan Sidwell  <nathan@codesourcery.com>

        PR target/28966
        PR target/29248
        * reload1.c (reload): Realign stack after it changes size.


-- 

nathan at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28966

[Bug target/28966] -maltivec -m32 causes the stack to be saved and restored even though there is no need for it

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |3.3.3 3.4.0 4.0.0 4.1.0
                   |                            |4.2.0
   Target Milestone|---                         |4.3.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28966

[Bug target/28966] -maltivec -m32 causes the stack to be saved and restored even though there is no need for it

[pthaugen at gcc dot gnu dot org]

------- Comment #4 from pthaugen at gcc dot gnu dot org  2007-10-31 19:48 -------
Subject: Bug 28966

Author: pthaugen
Date: Wed Oct 31 19:48:19 2007
New Revision: 129806

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=129806
Log:
2007-10-01  Pat Haugen  <pthaugen@us.ibm.com>

        Backport the following patches:

        2007-09-04  Daniel Jacobowitz  <dan@codesourcery.com>

        * config/rs6000/rs6000.c (rs6000_stack_info): Allocate space for the
        GOT pointer only if there is a constant pool.  Use the allocated space
        for SPE also.

        2006-12-21  Nathan Sidwell  <nathan@codesourcery.com>

        PR target/28966
        PR target/29248
        * reload1.c (reload): Realign stack after it changes size.


Modified:
    branches/ibm/gcc-4_1-branch/gcc/ChangeLog
    branches/ibm/gcc-4_1-branch/gcc/config/rs6000/rs6000.c
    branches/ibm/gcc-4_1-branch/gcc/reload1.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28966