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

Duncan P. N. Exon Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 3 18:33:34 PST 2020


dexonsmith accepted this revision.
dexonsmith 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?

(Not sure you need a test, but if it's valuable and easy to test, might as well; either way this LGTM...)


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