[PATCH] D135574: [MemCpyOpt] Update code comments for processMemSetMemCpyDependence. NFC

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 11 02:28:28 PDT 2022


bjope added a comment.

Alternative patch that aligns the code with the comments instead of updating the comments: D135653 <https://reviews.llvm.org/D135653>

Not sure which one is best or how to motivate the reordering (for out OOT target it might be better to do the memcpy first sometimes, but that is probably a more general scheduling thing rather than something this pass should need to think about).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135574



More information about the llvm-commits mailing list