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 PR68030/PR69710][RFC]Introduce a simple local CSE interface and use it in vectorizer

From: Jeff Law <law at redhat dot com>
To: "Bin.Cheng" <amker dot cheng at gmail dot com>, Richard Biener <richard dot guenther at gmail dot com>
Cc: Bin Cheng <Bin dot Cheng at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Tue, 21 Jun 2016 15:58:27 -0600
Subject: Re: [PATCH PR68030/PR69710][RFC]Introduce a simple local CSE interface and use it in vectorizer
Authentication-results: sourceware.org; auth=none
References: <DB5PR08MB1144AA1F16901D9CDDF918E4E7400 at DB5PR08MB1144 dot eurprd08 dot prod dot outlook dot com> <CAFiYyc1xt7UWondKbn74yC7Z+Trk4VE6+050zCk5VxtZ_zw-Vw at mail dot gmail dot com> <CAHFci2_fqF=MWdWJmE0Q8tpw1DNm+tdBp9J+Jg+g-Cjb5gZQ5Q at mail dot gmail dot com> <CAFiYyc1EFYR2sE_0vgTORnQJn7AN7vEBqYwaJonSTwHMRG_EaA at mail dot gmail dot com> <CAHFci2_CH1Fg3xmrOyc98xMbdEUzeJTkweJtBVMPkoAqqRtRxw at mail dot gmail dot com>

On 06/06/2016 05:23 AM, Bin.Cheng wrote:

Hi Jeff,
What's your opinion on this (and how to extend it to a region based
interface)?  I will update this patch for further review if you are
okay.

I've never pondered how to revamp DOM into a regional interface (thoughI have pondered how to revamp the threading bits into a regionalinterface -- without any real success).

Conceptually I think it's just a matter of setting an entry block/edge,computing all blocks dominated by that entry block/edge and limiting thewalker to only visiting blocks in that set. So the regional aspectwould live in domwalk.[ch].

We'd probably want to avoid all the threading bits in a regional walk --not because threading isn't useful, but sequencing the updates is a PITAand trying to do it in the middle of some other pass would just be a mess.


jeff

References:
- Re: [PATCH PR68030/PR69710][RFC]Introduce a simple local CSE interface and use it in vectorizer
  - From: Bin.Cheng

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