[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-12-14 Thread rearnsha at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

Richard Earnshaw  changed:

   What|Removed |Added

   Target Milestone|--- |6.3

[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-12-12 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

Thomas Preud'homme  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
  Known to work||6.2.1
 Resolution|--- |FIXED
  Known to fail|6.2.1   |6.2.0

--- Comment #11 from Thomas Preud'homme  ---
Fixed in all supported version of GCC (GCC 5 does not seem affected).

[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-12-12 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

--- Comment #10 from Thomas Preud'homme  ---
Author: thopre01
Date: Mon Dec 12 10:58:17 2016
New Revision: 243543

URL: https://gcc.gnu.org/viewcvs?rev=243543&root=gcc&view=rev
Log:
Fix PR77904: callee-saved register trashed when clobbering sp

2016-12-12 Thomas Preud'homme 

Backport from mainline
2016-11-22  Thomas Preud'homme  

gcc/
PR target/77904
* config/arm/arm.c (thumb1_compute_save_reg_mask): Mark frame pointer
in save register mask if it is needed.

gcc/testsuite/
PR target/77904
* gcc.target/arm/pr77904.c: New test.

Added:
branches/gcc-6-branch/gcc/testsuite/gcc.target/arm/pr77904.c
Modified:
branches/gcc-6-branch/gcc/ChangeLog
branches/gcc-6-branch/gcc/config/arm/arm.c
branches/gcc-6-branch/gcc/testsuite/ChangeLog

[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-12-10 Thread freddie_chopin at op dot pl
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

--- Comment #9 from Freddie Chopin  ---
Any chance for merging the fix to gcc-6 branch before gcc 6.3 would be
released?

[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-11-22 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

--- Comment #8 from Thomas Preud'homme  ---
(In reply to Freddie Chopin from comment #7)
> Could this be also backported to the gcc-6 branch? I guess there will be 6.3
> version (possibly before first 7.x version), so it would be nice to have
> this patch there (;

Yes that's planned once the patch has cooked in trunk for at least a week
without any report of a regression.

Best regards.

[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-11-22 Thread freddie_chopin at op dot pl
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

--- Comment #7 from Freddie Chopin  ---
Could this be also backported to the gcc-6 branch? I guess there will be 6.3
version (possibly before first 7.x version), so it would be nice to have this
patch there (;

[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-11-22 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

--- Comment #6 from Thomas Preud'homme  ---
Author: thopre01
Date: Tue Nov 22 10:57:55 2016
New Revision: 242694

URL: https://gcc.gnu.org/viewcvs?rev=242694&root=gcc&view=rev
Log:
Fix PR77904: callee-saved register trashed when clobbering sp

2016-11-22  Thomas Preud'homme  

Backport from mainline
2016-11-22  Thomas Preud'homme  

gcc/
PR target/77904
* config/arm/arm.c (thumb1_compute_save_reg_mask): Mark frame pointer
in save register mask if it is needed.

gcc/testsuite/
PR target/77904
* gcc.target/arm/pr77904.c: New test.


Added:
branches/ARM/embedded-6-branch/gcc/testsuite/gcc.target/arm/pr77904.c
Modified:
branches/ARM/embedded-6-branch/gcc/ChangeLog.arm
branches/ARM/embedded-6-branch/gcc/config/arm/arm.c
branches/ARM/embedded-6-branch/gcc/testsuite/ChangeLog.arm

[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-11-22 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

--- Comment #5 from Thomas Preud'homme  ---
Author: thopre01
Date: Tue Nov 22 10:44:29 2016
New Revision: 242693

URL: https://gcc.gnu.org/viewcvs?rev=242693&root=gcc&view=rev
Log:
2016-11-22  Thomas Preud'homme  

gcc/
PR target/77904
* config/arm/arm.c (thumb1_compute_save_reg_mask): Mark frame pointer
in save register mask if it is needed.

gcc/testsuite/
PR target/77904
* gcc.target/arm/pr77904.c: New test.

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

[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-11-02 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

Thomas Preud'homme  changed:

   What|Removed |Added

   Keywords||wrong-code
  Known to fail||6.2.1

--- Comment #4 from Thomas Preud'homme  ---
Add known fail for older releases and wrong-code tag.

[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-11-02 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

Thomas Preud'homme  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |thopre01 at gcc dot 
gnu.org

--- Comment #3 from Thomas Preud'homme  ---
Working on a patch.

[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-10-11 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

Thomas Preud'homme  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed|2016-10-10 00:00:00 |2016-10-11
 Ever confirmed|0   |1

--- Comment #2 from Thomas Preud'homme  ---
Changing the status to NEW since bug can be reproduced

[Bug target/77904] [ARM Cortex-M0] Frame pointer thrashes registers if assembly statements with "sp" clobber are used

2016-10-10 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77904

Thomas Preud'homme  changed:

   What|Removed |Added

   Last reconfirmed||2016-10-10
 CC||thopre01 at gcc dot gnu.org
  Known to fail||7.0

--- Comment #1 from Thomas Preud'homme  ---
I can reproduce the behavior mentioned on trunk.