[PATCH] D54740: [NewPM] fixing asserts on deleted loop in -print-after-all

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev marked an inline comment as done.
fedor.sergeev added inline comments.


================
Comment at: lib/Passes/StandardInstrumentations.cpp:58
     const LazyCallGraph::SCC *C = any_cast<const LazyCallGraph::SCC *>(IR);
-    assert(C);
+    // We get nullptr here when current SCC gets invalidated by the pass.
+    if (C == nullptr)
----------------
chandlerc wrote:
> fedor.sergeev wrote:
> > philip.pfaffe wrote:
> > > Is that still the case? You don't hook up the Invalidated callback below.
> > Hmm... indeed. I did it in the followup which handles -print-module-scope part and it has not been posted yet  :-(
> > Okey, let me clean this up once more.
> Did this happen?
not yet, lagging behind


Repository:
  rL LLVM

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

https://reviews.llvm.org/D54740