[PATCH] D90344: [POC][LoopVectorizer] Allow invariant loads/stores using masked gather/scatter for a scalable VF.
Sander de Smalen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 9 07:13:17 PST 2020
sdesmalen added inline comments.
================
Comment at: llvm/test/Transforms/LoopVectorize/scalable-loop-invariant-store-unpredicated-body-scalar-tail.ll:3
+; also pass the loop hint as a 'force-vector-width' flag to opt.
+; REQUIRES: aarch64-registered-target
+
----------------
ctetreau wrote:
> I suppose this is why you don't want to actually merge this currently? What happens if if it's not aarch64?
The `REQUIRES: aarch64-registered-target` is actually unnecessary, not sure why I thought this was needed.
This patch is probably simple enough to be reviewed as-is. The other POC patches I've split up into smaller NFC patches, but there is little to simplify here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90344/new/
https://reviews.llvm.org/D90344
More information about the llvm-commits
mailing list