[PATCH] D131606: [Loop Fusion] Sink/hoist memory instructions between loop fusion candidates

Whitney Tsang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 6 12:49:13 PDT 2022


Whitney accepted this revision as: Whitney.
Whitney added a comment.
This revision is now accepted and ready to land.

I agree with @congzhe that `CodeMoverUtils` fits better for the functionality that is added in this patch. I am ok that this being a 2-steps process, where you first add the functionality in loop fusion pass, and then improve `CodeMoverUtils`. 
Given that @aaronkintel added the TODOs, I can approve it, but please do improve `CodeMoverUtils`.


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

https://reviews.llvm.org/D131606



More information about the llvm-commits mailing list