[PATCH] D86925: [MachineSink] add one more profitable pattern for sinking

[ChenZheng via Phabricator via llvm-commits]

shchenz added a comment.



> Given adding that is going to take some time, you could push your patch for the cases we know are always profitable: when the uses are alive across the whole loop (line 636 in the current implementation.)

Thanks for this suggestion, updated this patch accordingly.
As expected, the improvement for the benchmark is gone now. : ( 
Will do the left part in following patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86925