[PATCH] D92625: Use isIgnored instead of checking pass name

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 3 18:37:15 PST 2020


aeubanks added a comment.

In D92625#2432814 <https://reviews.llvm.org/D92625#2432814>, @dexonsmith wrote:

> These are all (very minor) function changes, since `AnalysisManagerProxy` is in `isIgnored()`... that seems fine to me, but should it be tested somehow?

`*AnalysisManagerProxy` will never be passed through the interfaces modified in this change since it's an analysis pass, not a normal pass.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92625



More information about the llvm-commits mailing list