[PATCH] D113655: [IR] Allow all integer types for stepvector intrinsic

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 11 08:10:00 PST 2021


paulwalker-arm added a comment.

Hi @david-arm, the problem with this patch is you're enabling a route into code generation that was previously not available.  For example, the removed `stepvector_i1` test is something that we'll need to support and is likely to fail for SVE because `<vscale x 16 x i1>` is a legal type and thus will not use the same type promotion logic that your new `stepvector_nxv2i3` test covers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113655



More information about the llvm-commits mailing list