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

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 21 13:17:25 PDT 2017


rengolin added a comment.

In https://reviews.llvm.org/D32737#760457, @hfinkel wrote:

> > As I explained to Hal on his comment, that is correct but doesn't have the effect you're expecting.
>
> Which comment? FWIW, I didn't see a particular response.


Sorry, not yours, Sanjoy's:

https://reviews.llvm.org/D32737#inline-290542

> I'm also under the impression that this won't work because it would interfere with any ongoing vector calculations, spill code, etc. The point being that it is fixed for a particular process once the process begins (at least in practice).

Range calculations won't have to bother with the scale of the vector, as I'll try to explain on Sanjoy's reply later on.

Spill code may be problematic (variable stack), but that's a problem that I'm not sure we can fix with any notation.


https://reviews.llvm.org/D32737





More information about the llvm-commits mailing list