[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[pinskia@physics.uc.edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


pinskia@physics.uc.edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[3.2/? regression][m68k]    |[3.3/3.4 regression][m68k]
                   |m68k-elf-gcc -m5200 produces|m68k-elf-gcc -m5200 produces
                   |erroneous SImode set of     |erroneous SImode set of
                   |short varaible on stack     |short varaible on stack


------- Additional Comments From pinskia@physics.uc.edu  2003-05-26 01:32 -------
see comments for why I changed this to a regression.



------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[pinskia@physics.uc.edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


pinskia@physics.uc.edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |3.3.1

[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[falk at debian dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


falk at debian dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |WAITING


------- Additional Comments From falk at debian dot org  2003-06-21 11:07 -------
Maybe it's just me, but just from looking at the RTL I'm not certain there's
a bug here. Just writing to a short stack slot in SI mode looks just fine to
me, since shorts are padded to 4 bytes on the stack on m68k, or it might be
the result of store merging (except that we don't do that). So is there any
test case with observably wrong behaviour?

        Falk

[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[mmitchel at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.3.1                       |3.3.2


------- Additional Comments From mmitchel at gcc dot gnu dot org  2003-07-11 23:01 -------
Postoned until GCC 3.3.2; m68k is not a primary platform.

[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[dhazeghi at yahoo dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


dhazeghi at yahoo dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|dhazeghi at yahoo dot com   |
             Status|WAITING                     |RESOLVED
         Resolution|                            |INVALID


------- Additional Comments From dhazeghi at yahoo dot com  2003-08-24 18:20 -------
Dunno why this is in waiting, but it shouldn't be. Should be...

[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[dhazeghi at yahoo dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


dhazeghi at yahoo dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|INVALID                     |


------- Additional Comments From dhazeghi at yahoo dot com  2003-08-24 18:20 -------
Unconfirmed.

[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[pinskia at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |WAITING


------- Additional Comments From pinskia at gcc dot gnu dot org  2003-08-24 18:23 -------
Dara the reason why this was in waiting was this question: "So is there any
test case with observably wrong behaviour?"
So the question still stands.

[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[pinskia at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |RESOLVED
         Resolution|                            |INVALID


------- Additional Comments From pinskia at gcc dot gnu dot org  2003-09-05 01:51 -------
No feedback in 3 months (T-16 days).

[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[pirovano at logiclab dot it]

------- Additional Comments From pirovano at logiclab dot it  2004-03-08 15:57 -------
We are using rtems-4.6.0, opt "rtems-4.6" with  GNU gcc compiler 3.2.3 
on MCF5272 target and we encountered again a problem that seems PR 8309. 

This problem is very critical because all the code generated for  coldfire 
family is  unreliable   and to find the cause of possible malfunction is very 
hard. 

Here a short description of we encountered: 

We compile with -O4 -fno-force-mem
and a local variable declared as byte is managed in wrong mode.

Here we show the C-code and the assembler generated by compiler:

pInfo->InfoHw[0].Offset = offset+ p->Reg;
   move.b    -0x0B(a6),d0   ; offset(a6),d0
   move.b    d0,-0x0F(a6)   ; d0,-15(a6)   ;ok <Offset> is a Byte
   move.b    0x1(a2),d6     ; 1(a2),d6
   ext.w     d6
   add.l     -0x10(a6),d6    ;add.w doesn't exist in coldfire 
   move.w    d6,0x6(a1)      ;and using add.l it gets values
                             ;stored from -0x10(a6) to  -0x0d(a6) 


If we change the declaration of the local variable <offset>, from byte to 
short, the code becomes:

pInfo->InfoHw[0].Offset = offset+p->Reg
          move.b    0x1(a2),d7     ; 1(a2),d7
          ext.w     d7
          add.l     a5,d7          ; offset,d7
          move.w    d7,0x6(a1)     ; d7,EXTENDSFDF(a1)

In this case compiler uses a register for the local variable <offset> and then 
the problem disappear.



-- 


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

[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[kazu at cs dot umass dot edu]

------- Additional Comments From kazu at cs dot umass dot edu  2004-03-08 16:06 -------
Could you post a preprocessed testcase if it's not one of those that are
mentioned in this PR?

Thanks,


-- 


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

[Bug target/8309] [3.3/3.4 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[bernie at develer dot com]

------- Additional Comments From bernie at develer dot com  2004-05-28 17:08 -------
Reopened since still present on mainline: 
 http://gcc.gnu.org/ml/gcc/2004-05/msg01264.html 
 
Proposed patch: 
 http://gcc.gnu.org/ml/gcc/2004-05/msg01357.html 
 

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|INVALID                     |
   Target Milestone|3.3.2                       |3.5.0


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

[Bug rtl-optimization/8309] [3.3/3.4/3.5 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[giovannibajo at libero dot it]

------- Additional Comments From giovannibajo at libero dot it  2004-06-06 03:44 -------
Retargeting to 3.4.1, being a regression on that release branch.
Mark, there is a pending patch for this.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mmitchel at gcc dot gnu dot
                   |                            |org
   Target Milestone|3.5.0                       |3.4.1


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

[Bug rtl-optimization/8309] [3.3/3.4/3.5 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[mmitchel at gcc dot gnu dot org]

------- Additional Comments From mmitchel at gcc dot gnu dot org  2004-06-07 14:18 -------
I'm not comfortable approving this patch, given that changes to reload have such
great potential for causing problems on other targets.  However, if someone more
comfortable with reload approves the patch -- and tests it on at least one other
target where the chnage would apply -- it's OK for 3.4.1.

-- 


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

[Bug rtl-optimization/8309] [3.3/3.4/3.5 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[bernie at develer dot com]

------- Additional Comments From bernie at develer dot com  2004-06-10 08:03 -------
*** Bug 13312 has been marked as a duplicate of this bug. ***

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |Steve at Zook dot com


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

[Bug rtl-optimization/8309] [3.3/3.4/3.5 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[bernie at develer dot com]

------- Additional Comments From bernie at develer dot com  2004-06-12 01:15 -------
Fixed on mainline and 3.4-branch. 
 

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


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

[Bug rtl-optimization/8309] [3.3/3.4/3.5 regression][m68k] m68k-elf-gcc -m5200 produces erroneous SImode set of short varaible on stack

[giovannibajo at libero dot it]

------- Additional Comments From giovannibajo at libero dot it  2004-06-12 11:52 -------
Fixed by:

http://gcc.gnu.org/ml/gcc-cvs/2004-06/msg00463.html
http://gcc.gnu.org/ml/gcc-cvs/2004-06/msg00464.html

Bernie, you forgot mentioning the PR number in the ChangeLog. Can you please 
fix that?


-- 


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