[PATCH] D82243: [SVE] Remove calls to VectorType::getNumElements from Scalar

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 2 10:14:25 PDT 2020


sdesmalen accepted this revision.
sdesmalen added a comment.
This revision is now accepted and ready to land.

LGTM

I think this patch can land and doesn't affect the behaviour of the ACLE, as the code LowerMatrixIntrinsics only works on matrix intrinsics (always fixed width), and RewriteStatepointsForGC and Scalarizer are not passes run by default. The SROA changes seem safe because after D76720 <https://reviews.llvm.org/D76720> the code bails out early on for scalable vectors (with the exception of running mem2reg, but your changes don't affect that).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82243





More information about the llvm-commits mailing list