public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia
@ 2011-07-05 12:19 sebastian.huber@embedded-brains.de
  2011-07-06  6:27 ` [Bug target/49641] " sebastian.huber@embedded-brains.de
                   ` (11 more replies)
  0 siblings, 12 replies; 13+ messages in thread
From: sebastian.huber@embedded-brains.de @ 2011-07-05 12:19 UTC (permalink / raw)
  To: gcc-bugs

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

           Summary: [ARM] Wrong code for ARMv4T and stmia
           Product: gcc
           Version: 4.6.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: sebastian.huber@embedded-brains.de
            Target: arm-rtemseabi4.11


Created attachment 24689
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=24689
Sample code.

The attached source code generates the following assembler code:

00000000 <_Watchdog_Handler_initialization>:
   0:   2200            movs    r2, #0
   2:   4b06            ldr     r3, [pc, #24]   ; (1c
<_Watchdog_Handler_initialization+0x1c>)
   4:   601a            str     r2, [r3, #0]
   6:   4b06            ldr     r3, [pc, #24]   ; (20
<_Watchdog_Handler_initialization+0x20>)
   8:   601a            str     r2, [r3, #0]
   a:   4b06            ldr     r3, [pc, #24]   ; (24
<_Watchdog_Handler_initialization+0x24>)
   c:   601a            str     r2, [r3, #0]
   e:   4b06            ldr     r3, [pc, #24]   ; (28
<_Watchdog_Handler_initialization+0x28>)
  10:   1d19            adds    r1, r3, #4
  12:   c30e            stmia   r3!, {r1, r2, r3}
  14:   4b05            ldr     r3, [pc, #20]   ; (2c
<_Watchdog_Handler_initialization+0x2c>)
  16:   1d19            adds    r1, r3, #4
  18:   c30e            stmia   r3!, {r1, r2, r3}
  1a:   4770            bx      lr

In particular it generates: stmia r3!, {r1, r2, r3}

This instruction leads to UNPREDICTABLE behavior.  According to "ARM
Architecture Reference Manual, Issue I" we have in section "A7.1.57 STMIA",
"Operand restrictions":

"If <Rn> is specified in <registers>:
 o If <Rn> is the lowest-numbered register specified in <registers>, the
original value of <Rn> is stored.
 o Otherwise, the stored value of <Rn> is UNPREDICTABLE."

Since r3 is not the lowest-numbered register specified in {r1, r2, r3} the
stored value of r3 is UNPREDICTABLE.


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [ARM] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
@ 2011-07-06  6:27 ` sebastian.huber@embedded-brains.de
  2011-07-06  8:32 ` [Bug target/49641] [4.6 Regression] " rearnsha at gcc dot gnu.org
                   ` (10 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: sebastian.huber@embedded-brains.de @ 2011-07-06  6:27 UTC (permalink / raw)
  To: gcc-bugs

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

Sebastian Huber <sebastian.huber@embedded-brains.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to work|                            |4.4.6
      Known to fail|                            |4.6.1

--- Comment #1 from Sebastian Huber <sebastian.huber@embedded-brains.de> 2011-07-06 06:27:23 UTC ---
The command line is:

arm-rtemseabi4.11-gcc -mcpu=arm7tdmi-s -mfpu=vfp -mfloat-abi=soft -mthumb -O2
-c watchdog.i

We also get an assembler warning:

watchdog.s: Assembler messages:
watchdog.s:29: Warning: value stored for r3 is UNKNOWN
watchdog.s:32: Warning: value stored for r3 is UNKNOWN

Is the ARM7TDMI-S core still supported by GCC or is it too old?


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [4.6 Regression] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
  2011-07-06  6:27 ` [Bug target/49641] " sebastian.huber@embedded-brains.de
@ 2011-07-06  8:32 ` rearnsha at gcc dot gnu.org
  2011-08-01 14:32 ` rguenth at gcc dot gnu.org
                   ` (9 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: rearnsha at gcc dot gnu.org @ 2011-07-06  8:32 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Earnshaw <rearnsha at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |wrong-code
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011.07.06 08:31:43
                 CC|                            |bernds at gcc dot gnu.org
            Summary|[ARM] Wrong code for ARMv4T |[4.6 Regression] Wrong code
                   |and stmia                   |for ARMv4T and stmia
     Ever Confirmed|0                           |1

--- Comment #2 from Richard Earnshaw <rearnsha at gcc dot gnu.org> 2011-07-06 08:31:43 UTC ---
This is the regression caused by the new ldm/stm optimizers that were added for
4.6


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [4.6 Regression] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
  2011-07-06  6:27 ` [Bug target/49641] " sebastian.huber@embedded-brains.de
  2011-07-06  8:32 ` [Bug target/49641] [4.6 Regression] " rearnsha at gcc dot gnu.org
@ 2011-08-01 14:32 ` rguenth at gcc dot gnu.org
  2011-08-01 14:42 ` rguenth at gcc dot gnu.org
                   ` (8 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: rguenth at gcc dot gnu.org @ 2011-08-01 14:32 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.6.2


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [4.6 Regression] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
                   ` (2 preceding siblings ...)
  2011-08-01 14:32 ` rguenth at gcc dot gnu.org
@ 2011-08-01 14:42 ` rguenth at gcc dot gnu.org
  2011-08-02 13:11 ` bernds at gcc dot gnu.org
                   ` (7 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: rguenth at gcc dot gnu.org @ 2011-08-01 14:42 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P2


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [4.6 Regression] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
                   ` (3 preceding siblings ...)
  2011-08-01 14:42 ` rguenth at gcc dot gnu.org
@ 2011-08-02 13:11 ` bernds at gcc dot gnu.org
  2011-08-22 11:10 ` sebastian.huber@embedded-brains.de
                   ` (6 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: bernds at gcc dot gnu.org @ 2011-08-02 13:11 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #3 from Bernd Schmidt <bernds at gcc dot gnu.org> 2011-08-02 13:11:07 UTC ---
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01044.html
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02174.html


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [4.6 Regression] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
                   ` (4 preceding siblings ...)
  2011-08-02 13:11 ` bernds at gcc dot gnu.org
@ 2011-08-22 11:10 ` sebastian.huber@embedded-brains.de
  2011-09-12  9:23 ` sebastian.huber@embedded-brains.de
                   ` (5 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: sebastian.huber@embedded-brains.de @ 2011-08-22 11:10 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #4 from Sebastian Huber <sebastian.huber@embedded-brains.de> 2011-08-22 11:00:07 UTC ---
The patch from Bernd Schmidt fixes the problem in my case.  What is the status
of this patch?


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [4.6 Regression] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
                   ` (5 preceding siblings ...)
  2011-08-22 11:10 ` sebastian.huber@embedded-brains.de
@ 2011-09-12  9:23 ` sebastian.huber@embedded-brains.de
  2011-10-26 17:22 ` jakub at gcc dot gnu.org
                   ` (4 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: sebastian.huber@embedded-brains.de @ 2011-09-12  9:23 UTC (permalink / raw)
  To: gcc-bugs

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

Sebastian Huber <sebastian.huber@embedded-brains.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Target|arm-rtemseabi4.11           |arm-eabi-gcc
      Known to fail|                            |4.7.0

--- Comment #5 from Sebastian Huber <sebastian.huber@embedded-brains.de> 2011-09-12 09:15:48 UTC ---
It would be nice if this can be fixed for 4.6.2 and 4.7.0.


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [4.6 Regression] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
                   ` (6 preceding siblings ...)
  2011-09-12  9:23 ` sebastian.huber@embedded-brains.de
@ 2011-10-26 17:22 ` jakub at gcc dot gnu.org
  2011-11-16 17:20 ` rearnsha at gcc dot gnu.org
                   ` (3 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: jakub at gcc dot gnu.org @ 2011-10-26 17:22 UTC (permalink / raw)
  To: gcc-bugs

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.6.2                       |4.6.3

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-10-26 17:13:18 UTC ---
GCC 4.6.2 is being released.


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [4.6 Regression] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
                   ` (7 preceding siblings ...)
  2011-10-26 17:22 ` jakub at gcc dot gnu.org
@ 2011-11-16 17:20 ` rearnsha at gcc dot gnu.org
  2011-11-16 18:00 ` rearnsha at gcc dot gnu.org
                   ` (2 subsequent siblings)
  11 siblings, 0 replies; 13+ messages in thread
From: rearnsha at gcc dot gnu.org @ 2011-11-16 17:20 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #7 from Richard Earnshaw <rearnsha at gcc dot gnu.org> 2011-11-16 17:02:52 UTC ---
Author: rearnsha
Date: Wed Nov 16 17:02:44 2011
New Revision: 181416

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181416
Log:
2011-11-16  Richard Earnshaw  <rearnsha@arm.com>
        Bernd Schmidt <bernds@coudesourcery.com>
        Sebastian Huber <sebastian.huber@embedded-brains.de>

    PR target/49641
    * config/arm/arm.c (store_multiple_sequence): Avoid cases where
    the base reg is stored iff compiling for Thumb1.

    * gcc.target/arm/pr49641.c: New test.

Added:
    trunk/gcc/testsuite/gcc.target/arm/pr49641.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/arm/arm.c
    trunk/gcc/testsuite/ChangeLog


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [4.6 Regression] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
                   ` (8 preceding siblings ...)
  2011-11-16 17:20 ` rearnsha at gcc dot gnu.org
@ 2011-11-16 18:00 ` rearnsha at gcc dot gnu.org
  2011-11-16 18:23 ` rearnsha at gcc dot gnu.org
  2013-02-19 20:38 ` LpSolit at netscape dot net
  11 siblings, 0 replies; 13+ messages in thread
From: rearnsha at gcc dot gnu.org @ 2011-11-16 18:00 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #8 from Richard Earnshaw <rearnsha at gcc dot gnu.org> 2011-11-16 17:53:31 UTC ---
Author: rearnsha
Date: Wed Nov 16 17:53:28 2011
New Revision: 181418

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181418
Log:
2011-11-16  Richard Earnshaw  <rearnsha@arm.com>
        Bernd Schmidt <bernds@coudesourcery.com>
        Sebastian Huber <sebastian.huber@embedded-brains.de>

    PR target/49641
    * config/arm/arm.c (store_multiple_sequence): Avoid cases where
    the base reg is stored iff compiling for Thumb1.

    * gcc.target/arm/pr49641.c: New test.

Added:
    branches/gcc-4_6-branch/gcc/testsuite/gcc.target/arm/pr49641.c
Modified:
    branches/gcc-4_6-branch/gcc/ChangeLog
    branches/gcc-4_6-branch/gcc/config/arm/arm.c
    branches/gcc-4_6-branch/gcc/testsuite/ChangeLog


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [4.6 Regression] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
                   ` (9 preceding siblings ...)
  2011-11-16 18:00 ` rearnsha at gcc dot gnu.org
@ 2011-11-16 18:23 ` rearnsha at gcc dot gnu.org
  2013-02-19 20:38 ` LpSolit at netscape dot net
  11 siblings, 0 replies; 13+ messages in thread
From: rearnsha at gcc dot gnu.org @ 2011-11-16 18:23 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Earnshaw <rearnsha at gcc dot gnu.org> changed:

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

--- Comment #9 from Richard Earnshaw <rearnsha at gcc dot gnu.org> 2011-11-16 17:54:31 UTC ---
Fixed.


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Bug target/49641] [4.6 Regression] Wrong code for ARMv4T and stmia
  2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
                   ` (10 preceding siblings ...)
  2011-11-16 18:23 ` rearnsha at gcc dot gnu.org
@ 2013-02-19 20:38 ` LpSolit at netscape dot net
  11 siblings, 0 replies; 13+ messages in thread
From: LpSolit at netscape dot net @ 2013-02-19 20:38 UTC (permalink / raw)
  To: gcc-bugs


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

--- Comment #10 from jye2 at gcc dot gnu.org 2012-06-08 06:58:32 UTC ---
Author: jye2
Date: Fri Jun  8 06:58:25 2012
New Revision: 188327

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188327
Log:
Backport mainline r179607, r179979, r179980, r181416, r182014
2012-06-08  Joey Ye  <joey.ye@arm.com>

    Backport r182014 from mainline.
    2011-12-05  Kazu Hirata  <kazu@codesourcery.com>

    PR target/51408
    * config/arm/arm.md (*minmax_arithsi): Always require the else
    clause in the MINUS case.

    Backport r181416 from mainline.
    2011-11-16  Richard Earnshaw  <rearnsha@arm.com>
        Bernd Schmidt <bernds@coudesourcery.com>
        Sebastian Huber <sebastian.huber@embedded-brains.de>

    PR target/49641
    * config/arm/arm.c (store_multiple_sequence): Avoid cases where
    the base reg is stored iff compiling for Thumb1.

    Backport r179980 from mainline.
    2011-10-14  David Alan Gilbert  <david.gilbert@linaro.org>

    PR target/48126
    * config/arm/arm.c (arm_output_sync_loop): Move label before barrier.

    Backport r179979 from mainline.
    2011-10-14  David Alan Gilbert  <david.gilbert@linaro.org>

    * config/arm/arm.h (TARGET_HAVE_DMB_MCR): MCR Not available in Thumb1.

    Backport r179607 from mainline.
    2011-10-06  Bernd Schmidt  <bernds@codesourcery.com>

    PR target/49049
    * config/arm/arm.md (arm_subsi3_insn): Lose the last alternative.

Testsuites:
    Backport r182014 from mainline
    2011-12-05  Kazu Hirata  <kazu@codesourcery.com>

    PR target/51408
    * gcc.dg/pr51408.c: New.

    Backport r181416 from mainline
    2011-11-16  Richard Earnshaw  <rearnsha@arm.com>
        Bernd Schmidt <bernds@coudesourcery.com>
        Sebastian Huber <sebastian.huber@embedded-brains.de>

    PR target/49641
    * gcc.target/arm/pr49641.c: New test.

    Backport r179607 from mainline
    2011-10-06  Bernd Schmidt  <bernds@codesourcery.com>

    PR target/49049
    * gcc.c-torture/compile/pr49049.c: New test.


Added:
   
branches/ARM/embedded-4_6-branch/gcc/testsuite/gcc.c-torture/compile/pr49049.c
    branches/ARM/embedded-4_6-branch/gcc/testsuite/gcc.dg/pr51408.c
    branches/ARM/embedded-4_6-branch/gcc/testsuite/gcc.target/arm/pr49641.c
Modified:
    branches/ARM/embedded-4_6-branch/gcc/ChangeLog.arm
    branches/ARM/embedded-4_6-branch/gcc/config/arm/arm.c
    branches/ARM/embedded-4_6-branch/gcc/config/arm/arm.h
    branches/ARM/embedded-4_6-branch/gcc/config/arm/arm.md
    branches/ARM/embedded-4_6-branch/gcc/testsuite/ChangeLog.arm


^ permalink raw reply	[flat|nested] 13+ messages in thread
end of thread, other threads:[~2013-02-19 20:35 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-07-05 12:19 [Bug target/49641] New: [ARM] Wrong code for ARMv4T and stmia sebastian.huber@embedded-brains.de
2011-07-06  6:27 ` [Bug target/49641] " sebastian.huber@embedded-brains.de
2011-07-06  8:32 ` [Bug target/49641] [4.6 Regression] " rearnsha at gcc dot gnu.org
2011-08-01 14:32 ` rguenth at gcc dot gnu.org
2011-08-01 14:42 ` rguenth at gcc dot gnu.org
2011-08-02 13:11 ` bernds at gcc dot gnu.org
2011-08-22 11:10 ` sebastian.huber@embedded-brains.de
2011-09-12  9:23 ` sebastian.huber@embedded-brains.de
2011-10-26 17:22 ` jakub at gcc dot gnu.org
2011-11-16 17:20 ` rearnsha at gcc dot gnu.org
2011-11-16 18:00 ` rearnsha at gcc dot gnu.org
2011-11-16 18:23 ` rearnsha at gcc dot gnu.org
2013-02-19 20:38 ` LpSolit at netscape dot net

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).