[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