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

[Hal Finkel via Phabricator via llvm-commits]

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