[PATCH] D134648: [LangRef] Update text for vscale to be more flexible but maintain original intent.

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added a comment.

Although SME is my rational I'd rather keep things in the context of LLVM IR rather than any specific implementation.  It is not the intent of this change to represent multiple values of vscale concurrently and thus we do not need a way to query anything but the current value of vscale.  Please also remember that we're talking about more than just an intrinsic. `llvm.vscale()` returns the value of vscale from `<vscale x 2 x i64>`.  If we were to add another intrinsic, say `llvm.another_vscale()`, then that implies we'd need a new vector type of the form `<another_vscale x 2 x i64>`, which is not something we need for SME enablement.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134648/new/

https://reviews.llvm.org/D134648