[PATCH] D37648: [SLPVectorizer] Fix PR21780 Expansion of 256 bit vector loads fails to fold into shuffles

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 26 15:30:43 PDT 2017


hfinkel added a comment.

In https://reviews.llvm.org/D37648#879283, @dtemirbulatov wrote:

> In https://reviews.llvm.org/D37648#865562, @hfinkel wrote:
>
> > Thanks for working on this. Please post an RFC on llvm-dev about the intrinsic and how you intend to use it. You can reference this review, but we should make sure that we have design consensus before proceeding here.
>
>
> Here is the thread: http://lists.llvm.org/pipermail/llvm-dev/2017-September/117381.html


Yes. Are you planing on updating that thread? Last message I see says:

> but I am looking now at "speculation_marker" metadata and I am still not sure how to implement it better.

are you still looking at some other kind of metadata solution? Did you mean for "metadata" here to mean the intrinsic you're proposing?


https://reviews.llvm.org/D37648





More information about the llvm-commits mailing list