v.g.vassilev added inline comments.
================ Comment at: clang/lib/CodeGen/CodeGenModule.cpp:7226 + // assert(DeferredDeclsToEmit.empty() && + // "Should have emitted all decls deferred to emit."); assert(NewBuilder->DeferredDecls.empty() && ---------------- aaron.ballman wrote: > v.g.vassilev wrote: > > That should probably be a separate review with a testcase. > +1 -- the codegen code owners should weigh in on whether this is reasonable > as a temporary measure or not. https://reviews.llvm.org/D148435 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141215/new/ https://reviews.llvm.org/D141215 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
