[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