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).