public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug middle-end/55642] New: [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block")
@ 2012-12-10 17:24 Bernhard.Rosenkranzer at linaro dot org
2012-12-11 1:23 ` [Bug target/55642] " pinskia at gcc dot gnu.org
` (6 more replies)
0 siblings, 7 replies; 8+ messages in thread
From: Bernhard.Rosenkranzer at linaro dot org @ 2012-12-10 17:24 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55642
Bug #: 55642
Summary: [4.8 Regression] Invalid thumb code generated ("thumb
conditional instruction should be in IT block")
Classification: Unclassified
Product: gcc
Version: 4.8.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: middle-end
AssignedTo: unassigned@gcc.gnu.org
ReportedBy: Bernhard.Rosenkranzer@linaro.org
Created attachment 28914
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=28914
Test case (not yet reduced)
Building the attached preprocessed code with -std=gnu++11 -O2 -mthumb
-mcpu=cortex-a9 results in errors from the assembler:
[bero@localhost grouper-4.2.1-gcc-4.8 (master)]$
/opt/android-toolchain-trunk/bin/arm-linux-androideabi-g++ -std=gnu++11 -o
ARMAsmParser.o -O2 -mthumb -mcpu=cortex-a9 -c ARMAsmParser.i
/tmp/ccKYPdeC.s: Assembler messages:
/tmp/ccKYPdeC.s:7949: Error: thumb conditional instruction should be in IT
block -- `subne r2,r2,r3,asr#31'
/tmp/ccKYPdeC.s:21058: Error: thumb conditional instruction should be in IT
block -- `subne r2,r2,r3,asr#31'
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug target/55642] [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block")
2012-12-10 17:24 [Bug middle-end/55642] New: [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block") Bernhard.Rosenkranzer at linaro dot org
@ 2012-12-11 1:23 ` pinskia at gcc dot gnu.org
2012-12-11 10:25 ` kyrylo.tkachov at arm dot com
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: pinskia at gcc dot gnu.org @ 2012-12-11 1:23 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55642
Andrew Pinski <pinskia at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |assemble-failure
Target Milestone|--- |4.8.0
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug target/55642] [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block")
2012-12-10 17:24 [Bug middle-end/55642] New: [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block") Bernhard.Rosenkranzer at linaro dot org
2012-12-11 1:23 ` [Bug target/55642] " pinskia at gcc dot gnu.org
@ 2012-12-11 10:25 ` kyrylo.tkachov at arm dot com
2012-12-11 10:28 ` kyrylo.tkachov at arm dot com
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: kyrylo.tkachov at arm dot com @ 2012-12-11 10:25 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55642
--- Comment #1 from Kyrill Tkachov <kyrylo.tkachov at arm dot com> 2012-12-11 10:25:10 UTC ---
Created attachment 28924
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=28924
Reduced testcase
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug target/55642] [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block")
2012-12-10 17:24 [Bug middle-end/55642] New: [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block") Bernhard.Rosenkranzer at linaro dot org
2012-12-11 1:23 ` [Bug target/55642] " pinskia at gcc dot gnu.org
2012-12-11 10:25 ` kyrylo.tkachov at arm dot com
@ 2012-12-11 10:28 ` kyrylo.tkachov at arm dot com
2012-12-11 14:17 ` ktkachov at gcc dot gnu.org
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: kyrylo.tkachov at arm dot com @ 2012-12-11 10:28 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55642
Kyrill Tkachov <kyrylo.tkachov at arm dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |kyrylo.tkachov at arm dot
| |com
--- Comment #2 from Kyrill Tkachov <kyrylo.tkachov at arm dot com> 2012-12-11 10:28:11 UTC ---
Regression introduced with r193724.
The abssi2 patterns in thumb2.md output two instructions but the enclosing it
block thinks it's only one and therefore has the wrong number of t's (it
instead of itt).
Currently testing a patch to fix this.
Thanks,
Kyrill
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug target/55642] [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block")
2012-12-10 17:24 [Bug middle-end/55642] New: [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block") Bernhard.Rosenkranzer at linaro dot org
` (2 preceding siblings ...)
2012-12-11 10:28 ` kyrylo.tkachov at arm dot com
@ 2012-12-11 14:17 ` ktkachov at gcc dot gnu.org
2012-12-11 14:21 ` kyrylo.tkachov at arm dot com
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: ktkachov at gcc dot gnu.org @ 2012-12-11 14:17 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55642
--- Comment #3 from ktkachov at gcc dot gnu.org 2012-12-11 14:17:33 UTC ---
Author: ktkachov
Date: Tue Dec 11 14:17:28 2012
New Revision: 194398
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=194398
Log:
gcc/ChangeLog
2012-12-11 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
PR target/55642
* config/arm/thumb2.md (*thumb2_abssi2):
Set ce_count attribute to 2.
(*thumb2_neg_abssi2): Likewise.
gcc/testsuite/ChangeLog
2012-12-11 Kyrylo Tkachov <kyrylo.tkachov@arm.com>
PR target/55642
* gcc.target/arm/pr55642.c: New testcase.
Added:
trunk/gcc/testsuite/gcc.target/arm/pr55642.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/arm/thumb2.md
trunk/gcc/testsuite/ChangeLog
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug target/55642] [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block")
2012-12-10 17:24 [Bug middle-end/55642] New: [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block") Bernhard.Rosenkranzer at linaro dot org
` (3 preceding siblings ...)
2012-12-11 14:17 ` ktkachov at gcc dot gnu.org
@ 2012-12-11 14:21 ` kyrylo.tkachov at arm dot com
2012-12-11 14:30 ` kyrylo.tkachov at arm dot com
2012-12-18 14:46 ` ktkachov at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: kyrylo.tkachov at arm dot com @ 2012-12-11 14:21 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55642
--- Comment #4 from Kyrill Tkachov <kyrylo.tkachov at arm dot com> 2012-12-11 14:20:57 UTC ---
Should be fixed in trunk now
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug target/55642] [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block")
2012-12-10 17:24 [Bug middle-end/55642] New: [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block") Bernhard.Rosenkranzer at linaro dot org
` (4 preceding siblings ...)
2012-12-11 14:21 ` kyrylo.tkachov at arm dot com
@ 2012-12-11 14:30 ` kyrylo.tkachov at arm dot com
2012-12-18 14:46 ` ktkachov at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: kyrylo.tkachov at arm dot com @ 2012-12-11 14:30 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55642
--- Comment #5 from Kyrill Tkachov <kyrylo.tkachov at arm dot com> 2012-12-11 14:29:59 UTC ---
(In reply to comment #4)
> Should be fixed in trunk now
Can you please check that it works for you now?
http://gcc.gnu.org/viewcvs?view=revision&revision=194398
Thanks,
Kyrill
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug target/55642] [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block")
2012-12-10 17:24 [Bug middle-end/55642] New: [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block") Bernhard.Rosenkranzer at linaro dot org
` (5 preceding siblings ...)
2012-12-11 14:30 ` kyrylo.tkachov at arm dot com
@ 2012-12-18 14:46 ` ktkachov at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: ktkachov at gcc dot gnu.org @ 2012-12-18 14:46 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55642
ktkachov at gcc dot gnu.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |RESOLVED
CC| |ktkachov at gcc dot gnu.org
Resolution| |FIXED
--- Comment #6 from ktkachov at gcc dot gnu.org 2012-12-18 14:45:31 UTC ---
Fixed.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-12-18 14:46 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-12-10 17:24 [Bug middle-end/55642] New: [4.8 Regression] Invalid thumb code generated ("thumb conditional instruction should be in IT block") Bernhard.Rosenkranzer at linaro dot org
2012-12-11 1:23 ` [Bug target/55642] " pinskia at gcc dot gnu.org
2012-12-11 10:25 ` kyrylo.tkachov at arm dot com
2012-12-11 10:28 ` kyrylo.tkachov at arm dot com
2012-12-11 14:17 ` ktkachov at gcc dot gnu.org
2012-12-11 14:21 ` kyrylo.tkachov at arm dot com
2012-12-11 14:30 ` kyrylo.tkachov at arm dot com
2012-12-18 14:46 ` ktkachov at gcc dot gnu.org
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).