[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