[Bug target/45094] [arm] wrong instructions for dword move in some cases

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.5.2                       |4.5.3

--- Comment #8 from Richard Guenther <rguenth at gcc dot gnu.org> 2010-12-16 13:03:16 UTC ---
GCC 4.5.2 is being released, adjusting target milestone.

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[siarhei.siamashka at gmail dot com]

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

--- Comment #9 from Siarhei Siamashka <siarhei.siamashka at gmail dot com> 2010-12-18 15:43:39 UTC ---
Can this bug get a "[4.5 regression]" header please?

Even though the bug existed in gcc sources since 2007 (see the link in comment
1), the reported wrong-code problem itself was apparently latent until gcc 4.5,
and is not reproducible with older gcc versions.

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[siarhei.siamashka at gmail dot com]

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

--- Comment #10 from Siarhei Siamashka <siarhei.siamashka at gmail dot com> 2010-12-18 15:47:12 UTC ---
(In reply to comment #9)
> see the link in comment 1

Sorry, I mean the link in the original report from Akos:
http://repo.or.cz/w/official-gcc.git/commitdiff/f1225f6f

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[qiyao at gcc dot gnu.org]

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

--- Comment #11 from Yao Qi <qiyao at gcc dot gnu.org> 2010-12-27 08:35:21 UTC ---
Patch to backport it to 4.5 is here
http://gcc.gnu.org/ml/gcc-patches/2010-12/msg01858.html

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[dirtyepic at gentoo dot org]

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

Ryan Hill <dirtyepic at gentoo dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dirtyepic at gentoo dot org

--- Comment #12 from Ryan Hill <dirtyepic at gentoo dot org> 2010-12-28 00:47:14 UTC ---
Is that first hunk intentional?

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[qiyao at gcc dot gnu.org]

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

--- Comment #13 from Yao Qi <qiyao at gcc dot gnu.org> 2010-12-28 01:48:51 UTC ---
(In reply to comment #12)
> Is that first hunk intentional?

No.  It is from another PR backport, so it is redundant here.  Submit a clean
patch again http://gcc.gnu.org/ml/gcc-patches/2010-12/msg01873.html

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[pinskia at gcc dot gnu.org]

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

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|4.5.3                       |4.6.0

--- Comment #14 from Andrew Pinski <pinskia at gcc dot gnu.org> 2012-01-12 20:34:14 UTC ---
Fixed in 4.6.0.

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[ramana at gcc dot gnu dot org]

------- Comment #7 from ramana at gcc dot gnu dot org  2010-08-27 12:14 -------
can this be backported to the 4.5 branch please ? 


-- 

ramana at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.5.2


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

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[qiyao at gcc dot gnu dot org]

------- Comment #6 from qiyao at gcc dot gnu dot org  2010-08-18 12:34 -------
Subject: Bug 45094

Author: qiyao
Date: Wed Aug 18 12:33:43 2010
New Revision: 163338

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=163338
Log:
gcc/
        PR target/45094
        * config/arm/arm.c (output_move_double): Fix typo generating 
        instructions ('ldr'->'str').

gcc/testsuite/

        PR target/45094
        * gcc.target/arm/pr45094.c: New test.

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


-- 


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

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[qiyao at gcc dot gnu dot org]

------- Comment #5 from qiyao at gcc dot gnu dot org  2010-08-02 00:38 -------
(In reply to comment #4)
> > Post patch here http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00001.html
> 
> +                 output_asm_insn ("strr%?\t%H0, [%1, #4]", otherops);
> 
> s/strr/str/ ?
> 
Right, it is a typo.  Latest patch is posted here,
http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00018.html


-- 


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

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[akos dot pasztory at gmail dot com]

------- Comment #4 from akos dot pasztory at gmail dot com  2010-08-01 11:18 -------
> Post patch here http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00001.html

+                 output_asm_insn ("strr%?\t%H0, [%1, #4]", otherops);

s/strr/str/ ?


-- 


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

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[qiyao at gcc dot gnu dot org]

------- Comment #3 from qiyao at gcc dot gnu dot org  2010-08-01 08:44 -------
(In reply to comment #2)
> Created an attachment (id=21327)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21327&action=view) [edit]
> simplified testcase

Update this test case a little bit, with test commands.
Post patch here http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00001.html



-- 

qiyao at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |qiyao at gcc dot gnu dot org


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

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[siarhei dot siamashka at gmail dot com]

------- Comment #2 from siarhei dot siamashka at gmail dot com  2010-07-27 20:07 -------
Created an attachment (id=21327)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21327&action=view)
simplified testcase

Confirmed with gcc 4.5.0 here. Also tried but could not reproduce the problem
with gcc 4.4 (it just does not seem to be able to emit ldrd/strd instructions
with pre/post increment).


-- 


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

[Bug target/45094] [arm] wrong instructions for dword move in some cases

[ramana at gcc dot gnu dot org]

------- Comment #1 from ramana at gcc dot gnu dot org  2010-07-27 12:47 -------
Patches should be submitted to gcc-patches@gcc.gnu.org after having been
regression tested. Please also submit a testcase and appropriate Changelog
entries as documented here -  http://gcc.gnu.org/contribute.html#patches


Having said that however, this patch looks alright to me from the naked eye and
code inspection.


-- 

ramana at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
           Keywords|                            |wrong-code
   Last reconfirmed|0000-00-00 00:00:00         |2010-07-27 12:47:57
               date|                            |


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