[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