[Bug target/58460] Error: shift amount out of range 0 to 4 at operand 3 -- `sub sp,x1,x0,lsl 9'

2013-10-03 Thread mshawcroft at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58460

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 CC||mshawcroft at gcc dot gnu.org
   Target Milestone|--- |4.8.2
  Known to fail||4.8.0, 4.8.1

--- Comment #1 from mshawcroft at gcc dot gnu.org ---
r203157 | mshawcroft | 2013-10-03 11:54:36 +0100 (Thu, 03 Oct 2013) | 7 lines

[AArch64] Fix PR58460

PR58460, the add and sub shifted register instruction forms in AArch64
do not permit the stack register.  This patch removes k constraint
from the relevant patterns and adds reduced form of the test case.


[Bug target/58460] Error: shift amount out of range 0 to 4 at operand 3 -- `sub sp,x1,x0,lsl 9'

2013-10-04 Thread mshawcroft at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58460

--- Comment #2 from mshawcroft at gcc dot gnu.org ---
Author: mshawcroft
Date: Fri Oct  4 14:57:04 2013
New Revision: 203204

URL: http://gcc.gnu.org/viewcvs?rev=203204&root=gcc&view=rev
Log:
[AArch64] Fix PR58460.

2013-10-04  Marcus Shawcroft  

Backport from mainline.

PR target/58460
* config/aarch64/aarch64.md (*add__)
(*add__si_uxtw,*add_mul_imm_)
(*sub__)
(*sub__si_uxtw,*sub_mul_imm_, *sub_mul_imm_si_uxtw):
Remove k constraint.



Added:
branches/gcc-4_8-branch/gcc/testsuite/gcc.target/aarch64/pr58460.c
Modified:
branches/gcc-4_8-branch/gcc/ChangeLog
branches/gcc-4_8-branch/gcc/config/aarch64/aarch64.md
branches/gcc-4_8-branch/gcc/testsuite/ChangeLog


[Bug target/58460] Error: shift amount out of range 0 to 4 at operand 3 -- `sub sp,x1,x0,lsl 9'

2013-10-04 Thread mshawcroft at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58460

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #3 from mshawcroft at gcc dot gnu.org ---
Fix for 4.8.2+


[Bug other/60133] [4.8/4.9 Regression] wrong multiarch name on aarch64-linux-gnu

2014-03-19 Thread mshawcroft at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60133

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||mshawcroft at gcc dot gnu.org
 Resolution|--- |FIXED

--- Comment #3 from mshawcroft at gcc dot gnu.org ---
Fixed for 4.9 and back ported for 4.8.3 by the commits Jakub listed above.
Closed.


[Bug target/60580] aarch64 generates wrong code for __attribute__ ((optimize("no-omit-frame-pointer")))

2014-03-27 Thread mshawcroft at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60580

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

--- Comment #5 from mshawcroft at gcc dot gnu.org ---
Patch applied to trunk r208862


[Bug tree-optimization/46038] Vectorizer generates misaligned address for vld1 qn, [rn:alignment]

2015-03-13 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=46038

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||mshawcroft at gcc dot gnu.org
 Resolution|--- |INVALID

--- Comment #2 from mshawcroft at gcc dot gnu.org ---
Issue can't be reproduced in trunk or 4.9


[Bug target/59725] [4.9 Regression] ARM,AArch64 r206148 (PR tree-optimization/59544) caused regressions in pr52943

2014-01-21 Thread mshawcroft at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59725

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||mshawcroft at gcc dot gnu.org
 Resolution|--- |DUPLICATE

--- Comment #2 from mshawcroft at gcc dot gnu.org ---
This was resolved in r206319

*** This bug has been marked as a duplicate of bug 59651 ***


[Bug tree-optimization/59651] [4.9 Regression] Vectorizer failing to spot dependence causes incorrect code generation.

2014-01-21 Thread mshawcroft at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59651

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 CC||christophe.lyon at st dot com

--- Comment #10 from mshawcroft at gcc dot gnu.org ---
*** Bug 59725 has been marked as a duplicate of this bug. ***


[Bug target/63975] some of the builtin-arith-overflow* fail on aarch64

2015-10-29 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63975

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 CC||mshawcroft at gcc dot gnu.org
 Resolution|--- |FIXED

--- Comment #6 from mshawcroft at gcc dot gnu.org ---
This is already fixed.


[Bug testsuite/63971] Some of gcc.target/aarch64/test_frame_*.c tests fail now

2015-10-29 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63971

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||mshawcroft at gcc dot gnu.org
 Resolution|--- |FIXED

--- Comment #10 from mshawcroft at gcc dot gnu.org ---
Fixed by Tejas


[Bug target/64456] [AArch64] pr64252.c fail on aarch64_be

2015-10-29 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64456

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||mshawcroft at gcc dot gnu.org
 Resolution|--- |FIXED

--- Comment #3 from mshawcroft at gcc dot gnu.org ---
Resolved by Tejas/Richard


[Bug target/64149] -mno-lra bitrots, suggest to remove for GCC 5

2015-01-06 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64149

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 CC||mshawcroft at gcc dot gnu.org
   Assignee|unassigned at gcc dot gnu.org  |mshawcroft at gcc dot 
gnu.org

--- Comment #3 from mshawcroft at gcc dot gnu.org ---
Mine.


[Bug target/64224] [ARM] -mapcs -marm uses deprecated forms (as of ARMv7-A) of LDM in epilogues

2015-02-12 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64224

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 CC||mshawcroft at gcc dot gnu.org
 Resolution|--- |WONTFIX

--- Comment #4 from mshawcroft at gcc dot gnu.org ---
-mapcs is now deprecated in gcc5


[Bug target/61633] New: AArch64 SISD ASHR instruction split clobbers input register.

2014-06-27 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61633

Bug ID: 61633
   Summary: AArch64 SISD ASHR instruction split clobbers input
register.
   Product: gcc
   Version: 4.9.0
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: target
  Assignee: unassigned at gcc dot gnu.org
  Reporter: mshawcroft at gcc dot gnu.org

The FFT code in Android AAC codec at -O2 is incorrectly compiled resulting in
an infinite loop. I've failed to reduce the test case to something usable but
the problem in the back end is clearly the define_splits associated with
*aarch64_ashr_sisd_or_int_3.  Both of the splits inappropriately use one
of the input operand register as a scratch pad for the negated shift operand.


[Bug target/61633] AArch64 SISD ASHR instruction split clobbers input register.

2014-06-27 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61633

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Target||aarch64*-*-*
 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2014-06-27
   Assignee|unassigned at gcc dot gnu.org  |mshawcroft at gcc dot 
gnu.org
 Ever confirmed|0   |1


[Bug target/61633] AArch64 SISD ASHR instruction split clobbers input register.

2014-06-30 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61633

--- Comment #1 from mshawcroft at gcc dot gnu.org ---
Author: mshawcroft
Date: Mon Jun 30 07:54:59 2014
New Revision: 212137

URL: https://gcc.gnu.org/viewcvs?rev=212137&root=gcc&view=rev
Log:
[AArch64] Fix register clobber in, aarch64_ashr_sisd_or_int_3 split.

PR target/61633

The two split patterns associated with
aarch64_ashr_sisd_or_int_3 split the instruction into a NEG
followed by an SHL.  The split uses one of the input operands as a
scratch register to hold the output of the NEG resulting in register
corruption.

This patch adjusts the splits to use the output operand as the scratch
register.

2014-06-30  Marcus Shawcroft  

   PR target/61633
   * config/aarch64/aarch64.md (*aarch64_ashr_sisd_or_int_3):
   Add alternative; make early clobber.  Adjust both split patterns
   to use operand 0 as the working register.


Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/aarch64/aarch64.md


[Bug target/61633] AArch64 SISD ASHR instruction split clobbers input register.

2014-06-30 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61633

--- Comment #2 from mshawcroft at gcc dot gnu.org ---
Author: mshawcroft
Date: Mon Jun 30 11:58:18 2014
New Revision: 212143

URL: https://gcc.gnu.org/viewcvs?rev=212143&root=gcc&view=rev
Log:
[AArch64] Fix register clobber in, aarch64_ashr_sisd_or_int_3 split.

Backport from Mainline
2014-06-30  Marcus Shawcroft  

PR target/61633
* config/aarch64/aarch64.md (*aarch64_ashr_sisd_or_int_3):
Add alternative; make early clobber.  Adjust both split patterns
to use operand 0 as the working register.


Modified:
branches/gcc-4_9-branch/gcc/ChangeLog
branches/gcc-4_9-branch/gcc/config/aarch64/aarch64.md


[Bug target/61633] AArch64 SISD ASHR instruction split clobbers input register.

2014-06-30 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61633

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED
   Target Milestone|--- |4.9.1

--- Comment #3 from mshawcroft at gcc dot gnu.org ---
Fixed in trunk, back ported to 4.9 branch.


[Bug target/58622] With -fomit-frame-pointer, A64 does not generate post-decrement stores

2014-07-24 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58622

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from mshawcroft at gcc dot gnu.org ---
Issue resolved with Jiong's prologue / epilogue patch series committed to
trunk:

The relevant optimization commits are r212976, r212999, r213000.


[Bug target/58622] With -fomit-frame-pointer, A64 does not generate post-decrement stores

2014-07-24 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58622

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

   Target Milestone|--- |4.10.0


[Bug target/62262] aarch64 gcc generates invalid assembler

2014-09-05 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62262

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||mshawcroft at gcc dot gnu.org
 Resolution|--- |FIXED
   Target Milestone|--- |4.9.2

--- Comment #12 from mshawcroft at gcc dot gnu.org ---
This was resolved in trunk and 4.9 by Carrot.


[Bug target/63190] New: Assembler errors when building md5 code from fbb on aarch64

2014-09-05 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63190

Bug ID: 63190
   Summary: Assembler errors when building md5 code from fbb on
aarch64
   Product: gcc
   Version: 4.9.0
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: target
  Assignee: unassigned at gcc dot gnu.org
  Reporter: mshawcroft at gcc dot gnu.org

See the debian bug report here:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=758964

Venkat has patch in review here:
https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00320.html


[Bug target/63190] Assembler errors when building md5 code from fbb on aarch64

2014-09-05 Thread mshawcroft at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63190

mshawcroft at gcc dot gnu.org changed:

   What|Removed |Added

 Target||aarch64
   Priority|P3  |P2
 Status|UNCONFIRMED |NEW
   Last reconfirmed||2014-09-05
   Target Milestone|--- |4.9.2
 Ever confirmed|0   |1