NoQ marked an inline comment as done. NoQ added inline comments.
================ Comment at: clang/lib/StaticAnalyzer/Core/BugReporter.cpp:2185 } else { - assert(ErrorNode); - hash.AddPointer(GetCurrentOrPreviousStmt(ErrorNode)); ---------------- Szelethus wrote: > Why delete the assert? Because it's enforced by the type system these days. This is a method on `PathSensitiveBugReport` that always has an error node, as asserted in its constructor. I should have removed this assert in D66572. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67382/new/ https://reviews.llvm.org/D67382 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits