public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/59216] [ARM] negdi*extendsidi regression
  2013-11-20 16:48 [Bug target/59216] New: [ARM] negdi*extendsidi regression christophe.lyon at st dot com
@ 2013-11-20 16:48 ` christophe.lyon at st dot com
  2013-11-20 17:06 ` christophe.lyon at st dot com
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: christophe.lyon at st dot com @ 2013-11-20 16:48 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #1 from christophe.lyon at st dot com ---
Created attachment 31261
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=31261&action=edit
negsidi_test.c

testcase.


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug target/59216] New: [ARM] negdi*extendsidi regression
@ 2013-11-20 16:48 christophe.lyon at st dot com
  2013-11-20 16:48 ` [Bug target/59216] " christophe.lyon at st dot com
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: christophe.lyon at st dot com @ 2013-11-20 16:48 UTC (permalink / raw)
  To: gcc-bugs

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

            Bug ID: 59216
           Summary: [ARM] negdi*extendsidi regression
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: christophe.lyon at st dot com

Commit 197526 introduced a regression shown by the attached testcase.

When working, it should print 80000000, while it prints ffffffff80000000.

It works when compiling at -O0, and fails at -O2.
Tested on Chromebook.

The reduced testcase is derived from the GNU tar code & testsuite.


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug target/59216] [ARM] negdi*extendsidi regression
  2013-11-20 16:48 [Bug target/59216] New: [ARM] negdi*extendsidi regression christophe.lyon at st dot com
  2013-11-20 16:48 ` [Bug target/59216] " christophe.lyon at st dot com
@ 2013-11-20 17:06 ` christophe.lyon at st dot com
  2013-11-20 17:52 ` [Bug target/59216] [[4.9 Regression] ARM] " pinskia at gcc dot gnu.org
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: christophe.lyon at st dot com @ 2013-11-20 17:06 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #2 from christophe.lyon at st dot com ---
Basically, the working code does:
asrs    r3, r2, #31
negs    r2, r2
sbc.w   r3, r3, r3, lsl #1

while the failing one does:
negs    r2, r2
asrs    r3, r2, #31


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug target/59216] [[4.9 Regression] ARM] negdi*extendsidi regression
  2013-11-20 16:48 [Bug target/59216] New: [ARM] negdi*extendsidi regression christophe.lyon at st dot com
  2013-11-20 16:48 ` [Bug target/59216] " christophe.lyon at st dot com
  2013-11-20 17:06 ` christophe.lyon at st dot com
@ 2013-11-20 17:52 ` pinskia at gcc dot gnu.org
  2013-11-22 15:44 ` [Bug target/59216] [4.9 Regression] ARM " rearnsha at gcc dot gnu.org
  2013-11-22 15:45 ` rearnsha at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: pinskia at gcc dot gnu.org @ 2013-11-20 17:52 UTC (permalink / raw)
  To: gcc-bugs

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |wrong-code
   Target Milestone|---                         |4.9.0
            Summary|[ARM] negdi*extendsidi      |[[4.9 Regression] ARM]
                   |regression                  |negdi*extendsidi regression


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug target/59216] [4.9 Regression] ARM negdi*extendsidi regression
  2013-11-20 16:48 [Bug target/59216] New: [ARM] negdi*extendsidi regression christophe.lyon at st dot com
                   ` (2 preceding siblings ...)
  2013-11-20 17:52 ` [Bug target/59216] [[4.9 Regression] ARM] " pinskia at gcc dot gnu.org
@ 2013-11-22 15:44 ` rearnsha at gcc dot gnu.org
  2013-11-22 15:45 ` rearnsha at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: rearnsha at gcc dot gnu.org @ 2013-11-22 15:44 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #4 from Richard Earnshaw <rearnsha at gcc dot gnu.org> ---
Author: rearnsha
Date: Fri Nov 22 15:43:11 2013
New Revision: 205271

URL: http://gcc.gnu.org/viewcvs?rev=205271&root=gcc&view=rev
Log:
    PR target/59216

gcc/
    * arm.md (negdi_extendsidi): Fix invalid split.

gcc/testsuite/
    * gcc.target/arm/negdi-4.c: Delete invalid test.
    * gcc.dg/torture/pr59216.c: New test.

Added:
    trunk/gcc/testsuite/gcc.dg/torture/pr59216.c
Removed:
    trunk/gcc/testsuite/gcc.target/arm/negdi-4.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/arm/arm.md
    trunk/gcc/testsuite/ChangeLog


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug target/59216] [4.9 Regression] ARM negdi*extendsidi regression
  2013-11-20 16:48 [Bug target/59216] New: [ARM] negdi*extendsidi regression christophe.lyon at st dot com
                   ` (3 preceding siblings ...)
  2013-11-22 15:44 ` [Bug target/59216] [4.9 Regression] ARM " rearnsha at gcc dot gnu.org
@ 2013-11-22 15:45 ` rearnsha at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: rearnsha at gcc dot gnu.org @ 2013-11-22 15:45 UTC (permalink / raw)
  To: gcc-bugs

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

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

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

--- Comment #5 from Richard Earnshaw <rearnsha at gcc dot gnu.org> ---
Fixed.


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2013-11-22 15:45 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-11-20 16:48 [Bug target/59216] New: [ARM] negdi*extendsidi regression christophe.lyon at st dot com
2013-11-20 16:48 ` [Bug target/59216] " christophe.lyon at st dot com
2013-11-20 17:06 ` christophe.lyon at st dot com
2013-11-20 17:52 ` [Bug target/59216] [[4.9 Regression] ARM] " pinskia at gcc dot gnu.org
2013-11-22 15:44 ` [Bug target/59216] [4.9 Regression] ARM " rearnsha at gcc dot gnu.org
2013-11-22 15:45 ` rearnsha at gcc dot gnu.org

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).