[Bug regression/25871] New: TRAMPOLINE_TEMPLATE uses 32bit moves on 64bit code

[Bug regression/25871] New: TRAMPOLINE_TEMPLATE uses 32bit moves on 64bit code

[kumba at gentoo dot org]

In gcc/config/mips/mips.h, the TRAMPOLINE_TEMPLATE macro uses three 32bit move
statements, that when working with 64bit code, will cause problems.  The only
time this has been observed thus far was in filesystem code borrowed from grub
which relied heavily on nested functions.

A patch against trunk is attached, but this bug goes as far back as 3.3.x.


-- 
           Summary: TRAMPOLINE_TEMPLATE uses 32bit moves on 64bit code
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: regression
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: kumba at gentoo dot org


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

[Bug regression/25871] TRAMPOLINE_TEMPLATE uses 32bit moves on 64bit code

[kumba at gentoo dot org]

------- Comment #1 from kumba at gentoo dot org  2006-01-20 06:48 -------
Created an attachment (id=10683)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10683&action=view)
Use dmove/move where appropriate


-- 


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

[Bug regression/25871] TRAMPOLINE_TEMPLATE uses 32bit moves on 64bit code

[kumba at gentoo dot org]

------- Comment #2 from kumba at gentoo dot org  2006-01-20 06:54 -------
Created an attachment (id=10684)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10684&action=view)
Use dmove/move where appropriate

Typo in original, this is the correct version.


-- 

kumba at gentoo dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #10683|0                           |1
        is obsolete|                            |


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

[Bug target/25871] TRAMPOLINE_TEMPLATE uses 32bit moves on 64bit code

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2006-01-20 14:10 -------
Conifmred but this is actually not a regression from any versions of GCC (after
the EGCS split) that I can tell from as the source has not changed that much.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-01-20 14:10:16
               date|                            |


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

[Bug target/25871] TRAMPOLINE_TEMPLATE uses 32bit moves on 64bit code

[jsm28 at gcc dot gnu dot org]

------- Comment #4 from jsm28 at gcc dot gnu dot org  2006-04-26 08:39 -------
Subject: Bug 25871

Author: jsm28
Date: Wed Apr 26 08:38:58 2006
New Revision: 113272

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=113272
Log:
        From Bugzilla:
        2006-04-26  Joshua Kinard  <kumba@gentoo.org>

        PR target/25871
        * gcc/config/mips/mips.h (TRAMPOLINE_TEMPLATE): Avoid 32-bit moves
        with 64-bit pointers.

Modified:
    branches/csl-gxxpro-3_4-branch/ChangeLog.csl
    branches/csl-gxxpro-3_4-branch/gcc/config/mips/mips.h


-- 


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

[Bug target/25871] TRAMPOLINE_TEMPLATE uses 32bit moves on 64bit code

[jsm28 at gcc dot gnu dot org]

------- Comment #5 from jsm28 at gcc dot gnu dot org  2006-04-26 08:40 -------
Subject: Bug 25871

Author: jsm28
Date: Wed Apr 26 08:40:26 2006
New Revision: 113273

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=113273
Log:
        From Bugzilla:
        2006-04-26  Joshua Kinard  <kumba@gentoo.org>

        PR target/25871
        * gcc/config/mips/mips.h (TRAMPOLINE_TEMPLATE): Avoid 32-bit moves
        with 64-bit pointers.

Modified:
    branches/csl/sourcerygxx-4_1/ChangeLog.csl
    branches/csl/sourcerygxx-4_1/gcc/config/mips/mips.h


-- 


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

[Bug target/25871] TRAMPOLINE_TEMPLATE uses 32bit moves on 64bit code

[rsandifo at gcc dot gnu dot org]

------- Comment #6 from rsandifo at gcc dot gnu dot org  2006-04-28 22:36 -------
The patch looks good to me, thanks, and should be small enough to
avoid the need for a copyright assignment.  I'll apply it next week.


-- 

rsandifo at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |rsandifo at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2006-01-20 14:10:16         |2006-04-28 22:36:05
               date|                            |


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

[Bug target/25871] TRAMPOLINE_TEMPLATE uses 32bit moves on 64bit code

[rsandifo at gcc dot gnu dot org]

------- Comment #7 from rsandifo at gcc dot gnu dot org  2006-05-02 12:57 -------
Subject: Bug 25871

Author: rsandifo
Date: Tue May  2 12:57:52 2006
New Revision: 113460

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=113460
Log:
2006-05-02  Joshua Kinard  <kumba@gentoo.org>

        PR target/25871
        * gcc/config/mips/mips.h (TRAMPOLINE_TEMPLATE): Avoid 32-bit moves
        with 64-bit pointers.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/mips/mips.h


-- 


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

[Bug target/25871] TRAMPOLINE_TEMPLATE uses 32bit moves on 64bit code

[rsandifo at gcc dot gnu dot org]

------- Comment #8 from rsandifo at gcc dot gnu dot org  2006-05-02 13:02 -------
Fixed on mainline.


-- 

rsandifo at gcc dot gnu dot org changed:

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


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