[PATCH] D78987: [BPI][NFC] Reuse post dominantor tree from analysis manager when available

Evgeniy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 6 21:36:20 PDT 2020


ebrevnov added a comment.

Hi Alina,

I think you misread the patch. As Serguei already noted, PDT was unconditionally created by BPI even before my change. That means PDT is actually required analysis for BPI. But in order to support lazy BPI we still allow null PDT and do deferred creation of PDT if none was provided. Note also there are many other places in the code which explicitly creates BPI instance (not through analysis manager) and don't provide PDT as before.

Thanks
Evgeniy


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78987





More information about the llvm-commits mailing list