[PATCH] D67687: [NewPM][CodeGen] Introduce machine pass and machine pass manager

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D67687#2163365 <https://reviews.llvm.org/D67687#2163365>, @ychen wrote:

> In the MIR pipeline (what this patch is about), the IR is not changed and not ran. So all IR analyses results are preserved in MIR pipeline. Modifying MIR could only invalidate MIR analysis results but not IR analyses results.


This statement sounds too broad to me. For example, there may not be a 1:1 correspondence from MachineBasicBlock back to an IR BasicBlock, so any control flow analyses would be useless


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67687