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

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

In D130954#3693701 <https://reviews.llvm.org/D130954#3693701>, @michele.scandale wrote:

> 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?

Supporting this adds complexity, even if only a little.
Is the overhead of creating multiple pass managers measurable with your use case?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130954