* [Bug target/57735] ICE with -mtune=xscale (error: could not split insn) when building webkit
2013-06-27 15:56 [Bug c++/57735] New: ICE with -mtune=xscale (error: could not split insn) when building webkit Martin.Jansa at gmail dot com
@ 2013-06-27 16:43 ` ktkachov at gcc dot gnu.org
2013-06-28 8:20 ` ktkachov at gcc dot gnu.org
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: ktkachov at gcc dot gnu.org @ 2013-06-27 16:43 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57735
--- Comment #1 from ktkachov at gcc dot gnu.org ---
Confirmed with 4.8.1
Works with 4.7 and current trunk 4.9 though.
Fails for -mcpu=xscale , iwmmxt and iwmmxt2
Trying to reduce...
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/57735] ICE with -mtune=xscale (error: could not split insn) when building webkit
2013-06-27 15:56 [Bug c++/57735] New: ICE with -mtune=xscale (error: could not split insn) when building webkit Martin.Jansa at gmail dot com
2013-06-27 16:43 ` [Bug target/57735] " ktkachov at gcc dot gnu.org
@ 2013-06-28 8:20 ` ktkachov at gcc dot gnu.org
2013-06-28 10:27 ` ktkachov at gcc dot gnu.org
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: ktkachov at gcc dot gnu.org @ 2013-06-28 8:20 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57735
--- Comment #2 from ktkachov at gcc dot gnu.org ---
Created attachment 30406
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30406&action=edit
Reduced testcase
Reduced testcase, compile with -march=armv5te -marm -mtune=xscale
-mfloat-abi=soft -O1
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/57735] ICE with -mtune=xscale (error: could not split insn) when building webkit
2013-06-27 15:56 [Bug c++/57735] New: ICE with -mtune=xscale (error: could not split insn) when building webkit Martin.Jansa at gmail dot com
2013-06-27 16:43 ` [Bug target/57735] " ktkachov at gcc dot gnu.org
2013-06-28 8:20 ` ktkachov at gcc dot gnu.org
@ 2013-06-28 10:27 ` ktkachov at gcc dot gnu.org
2013-07-01 11:19 ` ktkachov at gcc dot gnu.org
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: ktkachov at gcc dot gnu.org @ 2013-06-28 10:27 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57735
ktkachov at gcc dot gnu.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Last reconfirmed| |2013-06-28
Ever confirmed|0 |1
--- Comment #3 from ktkachov at gcc dot gnu.org ---
Thus confirmed.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/57735] ICE with -mtune=xscale (error: could not split insn) when building webkit
2013-06-27 15:56 [Bug c++/57735] New: ICE with -mtune=xscale (error: could not split insn) when building webkit Martin.Jansa at gmail dot com
` (2 preceding siblings ...)
2013-06-28 10:27 ` ktkachov at gcc dot gnu.org
@ 2013-07-01 11:19 ` ktkachov at gcc dot gnu.org
2013-07-01 16:16 ` ktkachov at gcc dot gnu.org
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: ktkachov at gcc dot gnu.org @ 2013-07-01 11:19 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57735
--- Comment #5 from ktkachov at gcc dot gnu.org ---
(In reply to Mikael Pettersson from comment #4)
> The ICE on the reduced test case started with r186147 and stopped with
> r199188.
> They both touch the same code so I believe r199188 is a proper fix and not
> just coincidental. Backporting r199188 to current 4.8 branch fixes the ICE
> there too,
> but I haven't done a full bootstrap and regression test run on anything
> except x86_64 yet.
>
> r199188 did cause a regression (ICE on s390x), but the fix for that has
> already been backported to 4.8 branch (at r199813).
Thanks for investigating. I'll run a bootstrap on arm with the backported
patch.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/57735] ICE with -mtune=xscale (error: could not split insn) when building webkit
2013-06-27 15:56 [Bug c++/57735] New: ICE with -mtune=xscale (error: could not split insn) when building webkit Martin.Jansa at gmail dot com
` (3 preceding siblings ...)
2013-07-01 11:19 ` ktkachov at gcc dot gnu.org
@ 2013-07-01 16:16 ` ktkachov at gcc dot gnu.org
2013-07-01 18:30 ` mikpe at it dot uu.se
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: ktkachov at gcc dot gnu.org @ 2013-07-01 16:16 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57735
--- Comment #6 from ktkachov at gcc dot gnu.org ---
(In reply to Mikael Pettersson from comment #4)
> The ICE on the reduced test case started with r186147 and stopped with
> r199188.
> They both touch the same code so I believe r199188 is a proper fix and not
> just coincidental. Backporting r199188 to current 4.8 branch fixes the ICE
> there too,
> but I haven't done a full bootstrap and regression test run on anything
> except x86_64 yet.
>
> r199188 did cause a regression (ICE on s390x), but the fix for that has
> already been backported to 4.8 branch (at r199813).
After looking at it more closely: Mikael, are you sure the revisions are right?
It seems to me that r199813 is just the backport of r199188.
Can you please double-check the revision that fixes this current ICE?
Thanks,
Kyrill
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/57735] ICE with -mtune=xscale (error: could not split insn) when building webkit
2013-06-27 15:56 [Bug c++/57735] New: ICE with -mtune=xscale (error: could not split insn) when building webkit Martin.Jansa at gmail dot com
` (4 preceding siblings ...)
2013-07-01 16:16 ` ktkachov at gcc dot gnu.org
@ 2013-07-01 18:30 ` mikpe at it dot uu.se
2013-09-09 13:11 ` ktkachov at gcc dot gnu.org
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: mikpe at it dot uu.se @ 2013-07-01 18:30 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57735
--- Comment #7 from Mikael Pettersson <mikpe at it dot uu.se> ---
(In reply to ktkachov from comment #6)
> After looking at it more closely: Mikael, are you sure the revisions are
> right? It seems to me that r199813 is just the backport of r199188.
>
> Can you please double-check the revision that fixes this current ICE?
Oops. The ICE was fixed by r198462, r199188 is the followup. Sorry about that.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/57735] ICE with -mtune=xscale (error: could not split insn) when building webkit
2013-06-27 15:56 [Bug c++/57735] New: ICE with -mtune=xscale (error: could not split insn) when building webkit Martin.Jansa at gmail dot com
` (5 preceding siblings ...)
2013-07-01 18:30 ` mikpe at it dot uu.se
@ 2013-09-09 13:11 ` ktkachov at gcc dot gnu.org
2013-09-09 13:16 ` ktkachov at gcc dot gnu.org
2013-09-09 13:17 ` ktkachov at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: ktkachov at gcc dot gnu.org @ 2013-09-09 13:11 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57735
--- Comment #8 from ktkachov at gcc dot gnu.org ---
Author: ktkachov
Date: Mon Sep 9 13:11:45 2013
New Revision: 202397
URL: http://gcc.gnu.org/viewcvs?rev=202397&root=gcc&view=rev
Log:
[gcc/]
2013-09-09 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
PR target/57735
Backport from mainline
2013-04-30 Richard Sandiford <rsandifo@linux.vnet.ibm.com>
* explow.c (plus_constant): Pass "mode" to immed_double_int_const.
Use gen_int_mode rather than GEN_INT.
[gcc/testsuite]
2013-09-09 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
PR target/57735
* g++.dg/ext/pr57735.C: New test.
Added:
branches/gcc-4_8-branch/gcc/testsuite/g++.dg/ext/pr57735.C
Modified:
branches/gcc-4_8-branch/gcc/ChangeLog
branches/gcc-4_8-branch/gcc/explow.c
branches/gcc-4_8-branch/gcc/testsuite/ChangeLog
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/57735] ICE with -mtune=xscale (error: could not split insn) when building webkit
2013-06-27 15:56 [Bug c++/57735] New: ICE with -mtune=xscale (error: could not split insn) when building webkit Martin.Jansa at gmail dot com
` (6 preceding siblings ...)
2013-09-09 13:11 ` ktkachov at gcc dot gnu.org
@ 2013-09-09 13:16 ` ktkachov at gcc dot gnu.org
2013-09-09 13:17 ` ktkachov at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: ktkachov at gcc dot gnu.org @ 2013-09-09 13:16 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57735
--- Comment #9 from ktkachov at gcc dot gnu.org ---
Author: ktkachov
Date: Mon Sep 9 13:16:44 2013
New Revision: 202398
URL: http://gcc.gnu.org/viewcvs?rev=202398&root=gcc&view=rev
Log:
2013-09-09 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
PR target/57735
* g++.dg/ext/pr57735.C: New test.
Added:
trunk/gcc/testsuite/g++.dg/ext/pr57735.C
Modified:
trunk/gcc/testsuite/ChangeLog
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/57735] ICE with -mtune=xscale (error: could not split insn) when building webkit
2013-06-27 15:56 [Bug c++/57735] New: ICE with -mtune=xscale (error: could not split insn) when building webkit Martin.Jansa at gmail dot com
` (7 preceding siblings ...)
2013-09-09 13:16 ` ktkachov at gcc dot gnu.org
@ 2013-09-09 13:17 ` ktkachov at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: ktkachov at gcc dot gnu.org @ 2013-09-09 13:17 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57735
ktkachov at gcc dot gnu.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #10 from ktkachov at gcc dot gnu.org ---
Fixed in 4.8 branch. Testcase added in trunk.
^ permalink raw reply [flat|nested] 10+ messages in thread