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

[Renato Golin via Phabricator via llvm-commits]

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