This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH][RTL-ifcvt] Make non-conditional execution if-conversion more aggressive

From: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
To: Jeff Law <law at redhat dot com>, Steven Bosscher <stevenb dot gcc at gmail dot com>
Cc: Bernhard Reutner-Fischer <rep dot dot dot nop at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Tue, 11 Aug 2015 18:09:10 +0100
Subject: Re: [PATCH][RTL-ifcvt] Make non-conditional execution if-conversion more aggressive
Authentication-results: sourceware.org; auth=none
References: <559FBB13 dot 80009 at arm dot com> <CAC1BbcSjSYHd2j==dSwrjuMTrDvgnwrJJ2941k89aLEqnt49xg at mail dot gmail dot com> <55A388D3 dot 10506 at arm dot com> <55A3C53F dot 7080706 at arm dot com> <55B150D4 dot 5030909 at redhat dot com> <55B205F5 dot 3080005 at arm dot com> <55B28737 dot 5040403 at redhat dot com> <55B60525 dot 1060404 at arm dot com> <55B657B2 dot 70708 at redhat dot com> <55B65EFA dot 7010601 at arm dot com> <55B75601 dot 7050808 at arm dot com> <55BBAFAC dot 5020608 at redhat dot com> <CABu31nN3gvV3D1siu0B5cBh28J11oFrnXRUHvjV7x0Knk7ha0Q at mail dot gmail dot com> <55CA2B56 dot 4050804 at redhat dot com>


On 11/08/15 18:05, Jeff Law wrote:

On 08/09/2015 03:20 PM, Steven Bosscher wrote:

On Fri, Jul 31, 2015 at 7:26 PM, Jeff Law <law@redhat.com> wrote:

So there's a tight relationship between the implementation of
bbs_ok_for_cmove_arith and insn_valid_noce_process_p.  If there wasn't, then
we'd probably be looking to use note_stores and note_uses.

Perhaps I'm missing something, but what is wrong with using DF here
instead of note_stores/note_uses? All the info on refs/mods of
registers is available in the DF caches.

Nothing inherently wrong with using DF here.


I have reworked the patch to use FOR_EACH_INSN_DEF and FOR_EACH_INSN_USE
in bbs_ok_for_cmove_arith to extracts the refs/mods and it seems to work.
Is that what you mean by DF?
I'm doing some more testing and hope to post the updated version soon.

Thanks,
Kyrill

jeff

Follow-Ups:
- Re: [PATCH][RTL-ifcvt] Make non-conditional execution if-conversion more aggressive
  - From: Kyrill Tkachov

References:
- Re: [PATCH][RTL-ifcvt] Make non-conditional execution if-conversion more aggressive
  - From: Steven Bosscher
- Re: [PATCH][RTL-ifcvt] Make non-conditional execution if-conversion more aggressive
  - From: Jeff Law

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]