[PATCH] R600/SI: Use isProfitableToDupForIfCvt

Tom Stellard tom at stellard.net
Tue Mar 18 11:09:37 PDT 2014


On Tue, Mar 18, 2014 at 10:21:56AM -0700, Matt Arsenault wrote:
> On 03/18/2014 10:12 AM, Tom Stellard wrote:
> > Are these used outside of the IfConversion pass?  Because we don't use
> > IfConversion on SI.
> No. I assume that's because of the control flow intrinsics? I was 
> planning on trying something that would eventually remove them.
>

The main reason we don't run it on SI is that no one has implemented all
the required callbacks for predicating instructions or even the
Insert, Remove, and Analyze Branch functions.

I'm not sure whether or not the control flow intrinsics make it hard
or impossible to implement these, but I am interested to hear your idea
for removing them.

-Tom





More information about the llvm-commits mailing list