[PATCH] D134648: [LangRef] Update text for vscale to be more flexible but maintain original intent.
Amara Emerson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 27 02:33:00 PDT 2022
aemerson added a comment.
In D134648#3815329 <https://reviews.llvm.org/D134648#3815329>, @nikic wrote:
> I don't think we can allow this as long as vscale constant expressions are supported (these are currently represented via gep of scalable type). I am rather strongly opposed to having constants that are not in fact constant, so this change should only happen after a migration towards `llvm.vscale` intrinsics.
I haven't been following the scalable types story in a lot of detail, but doesn't disallowing GEP of scalable types imply that pointer calculations need to be done via decomposed integer arithmetic? Does this break for non-integral pointer types?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134648/new/
https://reviews.llvm.org/D134648
More information about the llvm-commits
mailing list