[PATCH] D105220: [ScalarizeMaskedMemIntrin] Use the element type to calculate alignment for gather/scatter when alignment operand is 0.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 30 13:14:33 PDT 2021


efriedma added a comment.

Instead of messing with this, can we just change the verifier to require that the alignment argument to masked_gather is nonzero?  (And teach autoupgrade to fix old code.)  This is just a lesser version of the same mess we used to have with other load/store operations.

If you're going to fix it this way, though, please also fix SelectionDAGBuilder.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105220



More information about the llvm-commits mailing list