[Bug rtl-optimization/88253] Inlining of function incorrectly deletes volatile register access when using XOR in avr-gcc

2018-12-17 Thread saaadhu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88253

Senthil Kumar Selvaraj  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED
   Assignee|segher at gcc dot gnu.org  |saaadhu at gcc dot 
gnu.org

--- Comment #8 from Senthil Kumar Selvaraj  ---
Fixed in trunk, and backported to gcc-8-branch and gcc-7-branch.

[Bug rtl-optimization/88253] Inlining of function incorrectly deletes volatile register access when using XOR in avr-gcc

2018-12-17 Thread saaadhu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88253

--- Comment #7 from Senthil Kumar Selvaraj  ---
Author: saaadhu
Date: Mon Dec 17 13:26:50 2018
New Revision: 267201

URL: https://gcc.gnu.org/viewcvs?rev=267201=gcc=rev
Log:
2018-12-17  Senthil Kumar Selvaraj  

Backport from trunk
2018-12-17  Senthil Kumar Selvaraj 


PR rtl-optimization/88253
* combine.c (combine_simplify_rtx): Test for side-effects before
substituting by zero.

2018-12-17  Senthil Kumar Selvaraj  

Backport from trunk
2018-12-17  Senthil Kumar Selvaraj 


PR rtl-optimization/88253
* gcc.target/avr/pr88253.c: New test.

Added:
branches/gcc-7-branch/gcc/testsuite/gcc.target/avr/pr88253.c
Modified:
branches/gcc-7-branch/gcc/ChangeLog
branches/gcc-7-branch/gcc/combine.c
branches/gcc-7-branch/gcc/testsuite/ChangeLog

[Bug rtl-optimization/88253] Inlining of function incorrectly deletes volatile register access when using XOR in avr-gcc

2018-12-17 Thread saaadhu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88253

--- Comment #6 from Senthil Kumar Selvaraj  ---
Author: saaadhu
Date: Mon Dec 17 12:07:26 2018
New Revision: 267199

URL: https://gcc.gnu.org/viewcvs?rev=267199=gcc=rev
Log:
2018-12-17  Senthil Kumar Selvaraj  

Backport from trunk
2018-12-17  Senthil Kumar Selvaraj 


PR rtl-optimization/88253
* combine.c (combine_simplify_rtx): Test for side-effects before
substituting by zero.

2018-12-17  Senthil Kumar Selvaraj  

Backport from trunk
2018-12-17  Senthil Kumar Selvaraj 


PR rtl-optimization/88253
* gcc.target/avr/pr88253.c: New test.

Added:
branches/gcc-8-branch/gcc/testsuite/gcc.target/avr/pr88253.c
Modified:
branches/gcc-8-branch/gcc/ChangeLog
branches/gcc-8-branch/gcc/combine.c
branches/gcc-8-branch/gcc/testsuite/ChangeLog

[Bug rtl-optimization/88253] Inlining of function incorrectly deletes volatile register access when using XOR in avr-gcc

2018-12-17 Thread saaadhu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88253

--- Comment #5 from Senthil Kumar Selvaraj  ---
Author: saaadhu
Date: Mon Dec 17 10:50:54 2018
New Revision: 267198

URL: https://gcc.gnu.org/viewcvs?rev=267198=gcc=rev
Log:
Fix PR 88253

gcc/ChangeLog:

PR rtl-optimization/88253
* combine.c (combine_simplify_rtx): Test for side-effects before
substituting by zero.

gcc/testsuite/ChangeLog:

PR rtl-optimization/88253
* gcc.target/avr/pr88253.c: New test.

Added:
trunk/gcc/testsuite/gcc.target/avr/pr88253.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/combine.c
trunk/gcc/testsuite/ChangeLog

[Bug rtl-optimization/88253] Inlining of function incorrectly deletes volatile register access when using XOR in avr-gcc

2018-12-14 Thread saaadhu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88253

Senthil Kumar Selvaraj  changed:

   What|Removed |Added

 CC||saaadhu at gcc dot gnu.org

--- Comment #4 from Senthil Kumar Selvaraj  ---
Oh shoot, just sent https://gcc.gnu.org/ml/gcc-patches/2018-12/msg01028.html :)

[Bug rtl-optimization/88253] Inlining of function incorrectly deletes volatile register access when using XOR in avr-gcc

2018-12-14 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88253

Segher Boessenkool  changed:

   What|Removed |Added

 Target|avr |avr-*-*
 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2018-12-14
 CC||segher at gcc dot gnu.org
   Host|Macos, Windows  |
   Assignee|unassigned at gcc dot gnu.org  |segher at gcc dot 
gnu.org
 Ever confirmed|0   |1

--- Comment #3 from Segher Boessenkool  ---
Mine.  I have a patch.

[Bug rtl-optimization/88253] Inlining of function incorrectly deletes volatile register access when using XOR in avr-gcc

2018-11-29 Thread westfw at westfw dot info
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88253

--- Comment #2 from William Westfield  ---
I did say that it still happens with 8.1...
However, the 5.4.0 version is the latest version supported by the vendor.

[Bug rtl-optimization/88253] Inlining of function incorrectly deletes volatile register access when using XOR in avr-gcc

2018-11-29 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88253

--- Comment #1 from Richard Biener  ---
Note GCC 5 is out of support, please see if you can reproduce this with a newer
version (GCC 7 and up).