[PATCH] D151550: DirectedGraph: Add a constructor that takes a list of Nodes.

Benjamin Kramer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 26 03:12:13 PDT 2023


bkramer added a comment.

Test case maybe?



================
Comment at: llvm/include/llvm/ADT/DirectedGraph.h:186
   DirectedGraph(DGraphType &&RHS) : Nodes(std::move(RHS.Nodes)) {}
+  DirectedGraph(NodeListTy Nodes) : Nodes(std::move(Nodes)) {}
   DGraphType &operator=(const DGraphType &G) {
----------------
Make it `explicit`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151550/new/

https://reviews.llvm.org/D151550



More information about the llvm-commits mailing list