[PATCH] D99569: [LoopVectorize] Fix bug where predicated loads/stores were dropped

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 19 05:59:10 PDT 2021


david-arm accepted this revision.
david-arm added a comment.
This revision is now accepted and ready to land.

LGTM! I think there is potentially a regression in the X86 test 'scalarize_and_sink_gather' in terms of number of generated lines of assembly, but the fix looks right for now. We could potentially improve the code quality in a future patch that looks for instructions with matching predicates and folds them into the same predicated block.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99569



More information about the llvm-commits mailing list