[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 11:02:30 PDT 2017
rengolin added a comment.
Just to be clear, I'm not *against* the idea of an intrinsic, nor I'm pushing this patch for any personal/professional agenda. I hope I have made that perfectly clear on my previous reviews on the same patches before.
I just want the best technical overall solution, and this particular one seems fine to me. I may be absolutely wrong, and that's perfectly fine, but we need a solution for this, even if we have to start with intrinsics and move to IR changes.
--renato
https://reviews.llvm.org/D32737
More information about the llvm-commits
mailing list