[clang] [WIP][analyzer] Refactor `ExplodedGraph::trim()` (PR #139939)

[Balazs Benics via cfe-commits]

================
@@ -2633,7 +2633,8 @@ BugPathGetter::BugPathGetter(const ExplodedGraph *OriginalGraph,
     assert(I->isValid() &&
            "We only allow BugReporterVisitors and BugReporter itself to "
            "invalidate reports!");
-    Nodes.emplace_back(I->getErrorNode());
+    if (const ExplodedNode *ErrNode = I->getErrorNode())
----------------
steakhal wrote:

I don't think the ErrorNode could be ever null.

https://github.com/llvm/llvm-project/pull/139939