[PATCH] D86864: [MachineSinking] sink more profitable loads

[ChenZheng via Phabricator via llvm-commits]

shchenz added a comment.

@efriedma @qcolombet haha, thanks for your review. Please take your time. ^-^ . I want to make the alias check simple(find all basic blocks in all paths from block `From` to block `To`), so I guard the dominance between `From` and `To` as `From` dominates `To` and `To` post `dominates` From. This will make us still miss some cases in which load instruction can be sunk. But it can make the implementation simple.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86864