[PATCH] D97299: [IR][SVE] Add new llvm.experimental.stepvector intrinsic

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 5 04:55:43 PST 2021


paulwalker-arm added a comment.

In D97299#2603806 <https://reviews.llvm.org/D97299#2603806>, @ctetreau wrote:

> If the ISD node is going to get an argument for the step, why not let the new intrinsic have this same argument?

@ctetreau:  We've followed the same route as for llvm.vscale() -> ISD::VSCALE so yes the code generation side is more complete/convenient.  Considering we're expecting `llvm.experimental.stepvector()` to be redundant once there's time to go push for the preferred ConstantVector solution we feel it's better to keep the intrinsic as simple as possible.


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

https://reviews.llvm.org/D97299



More information about the llvm-commits mailing list