[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