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

[Balazs Benics via cfe-commits]

================
@@ -4096,7 +4096,8 @@ std::string ExprEngine::DumpGraph(bool trim, StringRef Filename) {
 
 std::string ExprEngine::DumpGraph(ArrayRef<const ExplodedNode *> Nodes,
                                   StringRef Filename) {
-  std::unique_ptr<ExplodedGraph> TrimmedG(G.trim(Nodes));
+  TrimGraphWorklist Worklist{Nodes};
+  std::unique_ptr<ExplodedGraph> TrimmedG(G.trim(Worklist));
----------------
steakhal wrote:

Ah because it's now an in-out ref parameter. I don't understand why it needs to be an lvalue.

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