================
@@ -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));
----------------
NagyDonat wrote:

I pushed yet another small commit where I provide an "easy-to-call" overload 
for `trim` that takes an `ArrayRef` instead of a worklist: 
https://github.com/llvm/llvm-project/pull/139939/commits/e01e8bddd0959df81db7d34fbe800ed2a639a4af

However I'm also open to reverting [the commit where I start to consume the 
array of 
nodes](https://github.com/llvm/llvm-project/pull/139939/commits/c2a6891a417c854b2679236707a6a49f3a487b1a)
 if you think that this is premature optimiziation.

https://github.com/llvm/llvm-project/pull/139939
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to