[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 --- Comment #15 from Thomas Preud'homme thopre01 at gcc dot gnu.org --- (In reply to Thomas Preud'homme from comment #13) Thanks doko. I meant to do it today but lacked the time. As to the variation issue I've been experiencing, I'll create a new PR when I isolate the source of variation. It was just a red herring. Having no compiler in the path lead to HAVE_COMDAT_GROUP not to be defined which change the result of can_alias_cdtor and thus the number of times a pass is called. Making HAVE_COMDAT_GROUP be true, I can see there is still a difference in code generation but I guess it's likely a similar thing. No issue here. Best regards.
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 --- Comment #13 from Thomas Preud'homme thopre01 at gcc dot gnu.org --- Thanks doko. I meant to do it today but lacked the time. As to the variation issue I've been experiencing, I'll create a new PR when I isolate the source of variation. Best regards.
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 Matthias Klose doko at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #14 from Matthias Klose doko at gcc dot gnu.org --- backported, no regressions running the testsuite on x86_64-linux-gnu and arm-linux-gnueabihf.
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 --- Comment #12 from Matthias Klose doko at gcc dot gnu.org --- Author: doko Date: Tue Jun 23 10:08:11 2015 New Revision: 224833 URL: https://gcc.gnu.org/viewcvs?rev=224833root=gccview=rev Log: 2015-06-23 Matthias Klose d...@ubuntu.com PR target/66483 Backport from mainline r212178. 2014-06-30 Joseph Myers jos...@codesourcery.com * var-tracking.c (add_stores): Return instead of asserting if old and new values for conditional store are the same. Modified: branches/gcc-4_9-branch/gcc/ChangeLog branches/gcc-4_9-branch/gcc/var-tracking.c
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 Thomas Preud'homme thopre01 at gcc dot gnu.org changed: What|Removed |Added Assignee|thopre01 at gcc dot gnu.org|unassigned at gcc dot gnu.org --- Comment #11 from Thomas Preud'homme thopre01 at gcc dot gnu.org --- Thinking about this, there cannot be a regression by removing an assert so having the bug not fully reproducible should not be an issue. I'll continue looking for the cause of such variance but the backport of r212178 can be made right now.
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Target Milestone|4.9.4 |4.9.3
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 --- Comment #10 from Thomas Preud'homme thopre01 at gcc dot gnu.org --- There is something strange about this bug. I compiled an arm-linux-gnueabihf gcc only (using make all-gcc) twice, once with an arm-none-eabi cross binutils in the PATH and once without. The bug is reproduced with the former case but not the latter. I checked the objdump of object files in both case and most files have no difference, with a few showing at first glance only small offset difference. Could there be a dangling pointer issue at play? r212178 can be backported without issue but any full build of gcc for me falls in the latter case so I'm not confident with the testing.
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 --- Comment #9 from Thomas Preud'homme thopre01 at gcc dot gnu.org --- Patch can be backported without any changes and fixes the issue. I'll launch regression testing tomorrow and ask for it to be committed on 4.9 branch.
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 Lisandro Damián Nicanor Pérez Meyer perezmeyer at gmail dot com changed: What|Removed |Added CC||perezmeyer at gmail dot com --- Comment #5 from Lisandro Damián Nicanor Pérez Meyer perezmeyer at gmail dot com --- A very similar problem has been described in [bug] with even a fix which I believe it's part of gcc5 [fix] [bug] http://patchwork.openembedded.org/patch/92055/ [fix] https://gcc.gnu.org/viewcvs/gcc?view=revisionrevision=212178
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 --- Comment #6 from Lisandro Damián Nicanor Pérez Meyer perezmeyer at gmail dot com --- FWIW, it seems fixed in the linaro branch.
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 Thomas Preud'homme thopre01 at gcc dot gnu.org changed: What|Removed |Added CC||joseph at codesourcery dot com, ||thopre01 at gcc dot gnu.org --- Comment #7 from Thomas Preud'homme thopre01 at gcc dot gnu.org --- Hi Joseph, It seems you already fixed this bug in trunk. Is there any plan to backport to 4.9 branch? Best regards.
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 --- Comment #8 from joseph at codesourcery dot com joseph at codesourcery dot com --- I have no backport plans for this patch.
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 ktkachov at gcc dot gnu.org changed: What|Removed |Added Known to fail||4.9.3 --- Comment #3 from ktkachov at gcc dot gnu.org --- Reducing...
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Target Milestone|--- |4.9.4
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 --- Comment #1 from Matthias Klose doko at gcc dot gnu.org --- Created attachment 35738 -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=35738action=edit preprocessed source
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 ktkachov at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed||2015-06-10 CC||ktkachov at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #2 from ktkachov at gcc dot gnu.org --- Confirmed on latest 4.9 with command line options: -std=c++11 -g -O2 -fPIC -fstack-protector-strong -ffunction-sections -fno-exceptions -frandom-seed=0 -mfloat-abi=hard -mfpu=vfpv3-d16 -mthumb -march=armv7-a
[Bug target/66483] [4.9 Regression] ICE (in add_stores, at var-tracking.c:6000) on arm-linux-gnueabihf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66483 --- Comment #4 from ktkachov at gcc dot gnu.org --- (In reply to ktkachov from comment #3) Reducing... The initial submission already includes a reduced testcase. Had not noticed that