[PATCH] D66234: [MergedLoadStoreMotion] Sink stores to BB with more than 2 predecessors

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 4 10:34:11 PDT 2019


bjope accepted this revision.
bjope added a reviewer: lebedev.ri.
bjope added a comment.
This revision is now accepted and ready to land.

LGTM (apart from the comment about maybe saying something about the iteration order).

@lebedev.ri had comments about the test case earlier, so you could wait another day or two before landing this, to see if someone else got a different opinion (I don't know that much about this, but the code seem to do what you are aiming for afaict).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66234





More information about the llvm-commits mailing list