[PATCH] D72893: [NewPassManager] Add assertions when getting statefull cached analysis.

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 22 19:34:48 PDT 2020


asbirlea added inline comments.


================
Comment at: llvm/include/llvm/Analysis/CGSCCPassManager.h:856-858
+        auto *ResultFAMCP =
+            &CGAM.getResult<FunctionAnalysisManagerCGSCCProxy>(*C, CG);
+        ResultFAMCP->updateFAM(FAM);
----------------
chandlerc wrote:
> Check that it doesn't hit an assert failure, but I think you can remove this one.
Removing this hits the assertion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72893





More information about the llvm-commits mailing list