public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
From: "olegendo at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug target/51244] [SH] Inefficient conditional branch and code around T bit Date: Fri, 12 Oct 2012 00:41:00 -0000 [thread overview] Message-ID: <bug-51244-4-bzuFCapHIF@http.gcc.gnu.org/bugzilla/> (raw) In-Reply-To: <bug-51244-4@http.gcc.gnu.org/bugzilla/> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51244 --- Comment #55 from Oleg Endo <olegendo at gcc dot gnu.org> 2012-10-12 00:41:31 UTC --- Author: olegendo Date: Fri Oct 12 00:41:23 2012 New Revision: 192387 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=192387 Log: PR target/51244 * config/sh/sh.md (negsi_cond, negdi_cond, stack_protect_test): Remove get_t_reg_rtx when invoking gen_branch_true or gen_branch_false. (*zero_extend<mode>si2_compact): Convert to insn_and_split. Convert zero extensions of T bit stores to reg moves in splitter. Remove obsolete unnamed peephole2 that caught zero extensions after negc T bit stores. (*branch_true_eq, *branch_false_ne): Delete. (branch_true, branch_false): Convert insn to expander. Move actual insn logic to... (*cbranch_t): ...this new insn_and_split. Try to find preceding redundant T bit stores and tests and combine them with the conditional branch if possible in the splitter. (movrt_xor, *movt_movrt): New insn_and_split. * config/sh/predicates.md (cbranch_treg_value): New predicate. * config/sh/sh-protos.h (sh_eval_treg_value): Forward declare... * config/sh/sh.c (sh_eval_treg_value): ...this new function. (expand_cbranchsi4, expand_cbranchdi4): Remove get_t_reg_rtx when invoking gen_branch_true or gen_branch_false. PR target/51244 * gcc.target/sh/pr51244-13.c: New. * gcc.target/sh/pr51244-14.c: New. * gcc.target/sh/pr51244-15.c: New. * gcc.target/sh/pr51244-16.c: New. Added: trunk/gcc/testsuite/gcc.target/sh/pr51244-13.c trunk/gcc/testsuite/gcc.target/sh/pr51244-14.c trunk/gcc/testsuite/gcc.target/sh/pr51244-15.c trunk/gcc/testsuite/gcc.target/sh/pr51244-16.c Modified: trunk/gcc/ChangeLog trunk/gcc/config/sh/predicates.md trunk/gcc/config/sh/sh-protos.h trunk/gcc/config/sh/sh.c trunk/gcc/config/sh/sh.md trunk/gcc/testsuite/ChangeLog
next prev parent reply other threads:[~2012-10-12 0:41 UTC|newest] Thread overview: 89+ messages / expand[flat|nested] mbox.gz Atom feed top 2011-11-20 20:29 [Bug target/51244] New: SH Target: Inefficient conditional branch oleg.endo@t-online.de 2011-11-22 23:36 ` [Bug target/51244] " kkojima at gcc dot gnu.org 2011-12-27 22:03 ` oleg.endo@t-online.de 2011-12-27 23:17 ` oleg.endo@t-online.de 2011-12-28 0:42 ` oleg.endo@t-online.de 2011-12-28 4:57 ` oleg.endo@t-online.de 2011-12-28 16:07 ` oleg.endo@t-online.de 2011-12-28 22:30 ` kkojima at gcc dot gnu.org 2011-12-30 22:18 ` oleg.endo@t-online.de 2012-02-26 23:36 ` olegendo at gcc dot gnu.org 2012-03-02 21:57 ` olegendo at gcc dot gnu.org 2012-03-03 12:32 ` olegendo at gcc dot gnu.org 2012-03-04 17:25 ` olegendo at gcc dot gnu.org 2012-03-05 23:13 ` olegendo at gcc dot gnu.org 2012-03-05 23:38 ` olegendo at gcc dot gnu.org 2012-03-06 8:28 ` olegendo at gcc dot gnu.org 2012-03-06 8:50 ` kkojima at gcc dot gnu.org 2012-03-06 9:48 ` olegendo at gcc dot gnu.org 2012-03-06 10:36 ` kkojima at gcc dot gnu.org 2012-03-06 10:38 ` kkojima at gcc dot gnu.org 2012-03-06 10:39 ` kkojima at gcc dot gnu.org 2012-03-06 10:40 ` kkojima at gcc dot gnu.org 2012-03-06 11:30 ` olegendo at gcc dot gnu.org 2012-03-06 23:43 ` olegendo at gcc dot gnu.org 2012-03-08 1:26 ` olegendo at gcc dot gnu.org 2012-03-08 11:12 ` kkojima at gcc dot gnu.org 2012-03-08 11:15 ` kkojima at gcc dot gnu.org 2012-03-08 11:17 ` kkojima at gcc dot gnu.org 2012-03-09 0:27 ` olegendo at gcc dot gnu.org 2012-03-09 1:45 ` kkojima at gcc dot gnu.org 2012-03-09 8:41 ` kkojima at gcc dot gnu.org 2012-03-09 10:02 ` olegendo at gcc dot gnu.org 2012-03-09 10:37 ` kkojima at gcc dot gnu.org 2012-03-11 13:18 ` olegendo at gcc dot gnu.org 2012-03-15 8:11 ` kkojima at gcc dot gnu.org 2012-03-20 1:46 ` olegendo at gcc dot gnu.org 2012-03-20 2:33 ` kkojima at gcc dot gnu.org 2012-03-20 20:41 ` olegendo at gcc dot gnu.org 2012-05-07 20:53 ` olegendo at gcc dot gnu.org 2012-05-08 21:43 ` olegendo at gcc dot gnu.org 2012-06-30 12:01 ` olegendo at gcc dot gnu.org 2012-07-02 19:24 ` olegendo at gcc dot gnu.org 2012-07-08 15:03 ` olegendo at gcc dot gnu.org 2012-07-23 22:58 ` olegendo at gcc dot gnu.org 2012-07-23 23:29 ` olegendo at gcc dot gnu.org 2012-07-26 0:20 ` olegendo at gcc dot gnu.org 2012-07-30 6:46 ` olegendo at gcc dot gnu.org 2012-08-09 15:55 ` olegendo at gcc dot gnu.org 2012-08-12 22:47 ` olegendo at gcc dot gnu.org 2012-08-20 20:51 ` olegendo at gcc dot gnu.org 2012-08-30 22:54 ` olegendo at gcc dot gnu.org 2012-08-31 10:55 ` kkojima at gcc dot gnu.org 2012-08-31 15:50 ` olegendo at gcc dot gnu.org 2012-09-04 8:03 ` olegendo at gcc dot gnu.org 2012-09-23 21:36 ` [Bug target/51244] [SH] Inefficient conditional branch and code around T bit olegendo at gcc dot gnu.org 2012-09-23 21:42 ` olegendo at gcc dot gnu.org 2012-10-03 21:39 ` olegendo at gcc dot gnu.org 2012-10-12 0:41 ` olegendo at gcc dot gnu.org [this message] 2012-10-15 22:08 ` olegendo at gcc dot gnu.org 2012-11-03 12:01 ` olegendo at gcc dot gnu.org 2013-07-18 16:11 ` laurent.alfonsi at st dot com 2013-07-18 16:12 ` laurent.alfonsi at st dot com 2013-07-20 14:38 ` olegendo at gcc dot gnu.org 2013-07-23 8:21 ` laurent.alfonsi at st dot com 2013-07-27 19:28 ` olegendo at gcc dot gnu.org 2013-07-28 8:51 ` olegendo at gcc dot gnu.org 2013-07-28 12:26 ` olegendo at gcc dot gnu.org 2013-07-31 21:46 ` olegendo at gcc dot gnu.org 2013-08-23 0:13 ` olegendo at gcc dot gnu.org 2013-08-23 0:25 ` kkojima at gcc dot gnu.org 2013-09-24 22:43 ` olegendo at gcc dot gnu.org 2013-10-03 22:50 ` olegendo at gcc dot gnu.org 2013-10-12 20:47 ` olegendo at gcc dot gnu.org 2013-10-12 21:26 ` olegendo at gcc dot gnu.org 2013-12-05 17:54 ` olegendo at gcc dot gnu.org 2013-12-06 10:47 ` olegendo at gcc dot gnu.org 2014-05-10 20:19 ` olegendo at gcc dot gnu.org 2014-05-16 22:55 ` olegendo at gcc dot gnu.org 2014-09-13 18:48 ` olegendo at gcc dot gnu.org 2014-11-22 15:07 ` olegendo at gcc dot gnu.org 2014-11-22 15:50 ` olegendo at gcc dot gnu.org 2014-11-22 16:08 ` olegendo at gcc dot gnu.org 2014-12-01 6:50 ` olegendo at gcc dot gnu.org 2014-12-17 22:53 ` olegendo at gcc dot gnu.org 2014-12-17 23:08 ` olegendo at gcc dot gnu.org 2014-12-17 23:15 ` olegendo at gcc dot gnu.org 2014-12-24 21:56 ` olegendo at gcc dot gnu.org 2015-01-24 13:06 ` olegendo at gcc dot gnu.org 2015-03-01 19:16 ` olegendo at gcc dot gnu.org
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=bug-51244-4-bzuFCapHIF@http.gcc.gnu.org/bugzilla/ \ --to=gcc-bugzilla@gcc.gnu.org \ --cc=gcc-bugs@gcc.gnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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).