[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