[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 Sebastian Huber changed: What|Removed |Added Target|arm-rtemseabi4.11 |arm-eabi-gcc --- Comment #5 from Sebast

[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 --- Comment #6 from Sebastian Huber 2011-10-18 14:19:55 UTC --- Created attachment 25543 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25543 arm-eabi-g++ -march=armv5t -mthumb -Os -S compiler1.test.ii -o compiler1.test.eabi.GCC-4.5.4.Os.s

[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 Ramana Radhakrishnan changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|unassi

[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 --- Comment #8 from Ramana Radhakrishnan 2011-10-20 09:07:36 UTC --- Author: ramana Date: Thu Oct 20 09:07:30 2011 New Revision: 180240 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=180240 Log: 2011-10-20 Ramana Radhakrishnan

[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 --- Comment #9 from Ramana Radhakrishnan 2011-10-20 09:24:10 UTC --- Author: ramana Date: Thu Oct 20 09:24:06 2011 New Revision: 180241 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=180241 Log: Backport from mainline fix for PR target/5

[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 Ramana Radhakrishnan changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|

[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 --- Comment #11 from Sebastian Huber 2011-10-20 11:07:09 UTC --- Thank you very much. With this change the GCC 4.6.2-RC-20111019 produces now correct code in this case. I know understand why the unused volatile registers are saved and restored.

[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 --- Comment #12 from amker at gcc dot gnu.org 2012-06-12 02:50:37 UTC --- Author: amker Date: Tue Jun 12 02:50:34 2012 New Revision: 188416 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188416 Log: Backport r180240 from mainline 20

[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 --- Comment #1 from Sebastian Huber 2011-08-17 08:53:00 UTC --- Created attachment 25029 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25029 arm-rtemseabi4.11-g++ -march=armv5t -mthumb -Os -S compiler1.test.ii -o compiler1.test.eabi.Os.s

[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 --- Comment #2 from Sebastian Huber 2011-08-17 08:54:55 UTC --- Created attachment 25030 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25030 arm-rtemseabi4.11-g++ -march=armv5t -mthumb -O2 -S compiler1.test.ii -o compiler1.test.eabi.O2.s

[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 Ramana Radhakrishnan changed: What|Removed |Added Keywords||wrong-code Status|UNCO

[Bug target/50106] [ARM] Wrong code with -march=armv5t -mthumb -Os

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50106 --- Comment #4 from Sebastian Huber 2011-08-22 09:43:39 UTC --- Yes, this patch fixes the problem. It is still not clear to me why we save the volatile registers r0, r1, and r2 at all. Also we restore r1, r2, and r3. Does this make sense? I t