[PATCH] D71696: [CodeGen] WIP MachinePostRAUpdater

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 20 11:11:17 PST 2019


SjoerdMeijer added a comment.

Good stuff!

I haven't looked at details yet, but wanted to ask a high-level dumb question first. Essentially, this is DCE on MachineFunctions/MachineLoops. Just floating an idea, and you can correct me where I go wrong. Do you perhaps see any benefits in making this is a separate pass? I was thinking that by separating the concerns (i.e. LowOverhead loops, and cleaning-up and DCE), we can have a clean-up pass running after low-overhead loops, with possibly the benefit being that low-overhead loops does not have to deal with any of the cleanup and bookkeeping.


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

https://reviews.llvm.org/D71696





More information about the llvm-commits mailing list