[PATCH] D130954: [DeadArgElim] Clear state before returning (NFC).

Michele Scandale via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 2 08:36:41 PDT 2022


michele.scandale added a comment.

In D130954#3693243 <https://reviews.llvm.org/D130954#3693243>, @nikic wrote:

> Is there an RFC (or other context) for this patch series? It's not entirely obvious to me why we want to go out of the way to allow multiple run() calls.

I would think that a pass manager instance can be reused for processing different modules (that's why there might be multiple calls to `run`). This is something that so far has been working overall fine with the legacy pass manager. Is there something fundamentally different in the new pass manager infrastructure for which the same pass manager cannot be used for more than one module?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130954



More information about the llvm-commits mailing list