[PATCH] D32737: [Constants][SVE] Represent the runtime length of a scalable vector

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 20 17:15:41 PDT 2017


hfinkel added a comment.

In https://reviews.llvm.org/D32737#760292, @sanjoy wrote:

> In https://reviews.llvm.org/D32737#760289, @lattner wrote:
>
> > What are the semantics of select when the two vectors have different width?  Does store do a memory allocation?
>
>
> Maybe I misunderstood, but won't those selects be ill-typed?


That is my understanding as well; the model is that there is one underlying vector width, we just don't know at compile time what it is. In any case, is there an overall LangRef patch? It might be best to clear up the semantics with an overall patch (even if we commit the changes in pieces along with the implementation).


https://reviews.llvm.org/D32737





More information about the llvm-commits mailing list