[PATCH] D19984: [LV] Handle RAW dependences in interleaved access analysis

Adam Nemet via llvm-commits llvm-commits at lists.llvm.org
Wed May 11 15:21:12 PDT 2016


anemet added a comment.

In http://reviews.llvm.org/D19984#424731, @mssimpso wrote:

> Hi Silviu, thanks for the comments.
>
> > I think non-zero dependences would have already been rejected by LAA. Would this this be the reason why it is correct to only look at the zero distance ones?
>
>
> Yes, that should be the case!


Why are non-zero forward deps rejected by LAA?  Because of the HW store-to-load forwarding case?  I think that that is only a performance consideration and it's only on conditionally.

I think that we should probably take the time and review the soundness of these code motions with respect to interleaved access vectorization {RAW, WAR, WAW} x {loop-independent, loop-carried}.  I've only looked at the LAA aspects of this feature so far but I am a bit worried about this code now.

Matt, do you think you can do this?

I am also thinking if we should disable this feature in the meantime?!


http://reviews.llvm.org/D19984





More information about the llvm-commits mailing list