[PATCH] D119571: [funcattrs] use DominatorTree to improve noreturn

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 11 11:58:34 PST 2022


nickdesaulniers added a comment.

One thing I was curious of; does this recalculate a new `DominatorTree` for each function?

I noticed some other usages of `DominatorTree` first would query `Pass::getAnalysisIfAvailable<DominatorTreeWrapperPass>())`. Should I be doing that, too? (will require plumbing the `Pass` object through a few static functions, but isn't too bad).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119571



More information about the llvm-commits mailing list