[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