aprantl accepted this revision. aprantl added inline comments. This revision is now accepted and ready to land.
================ Comment at: lldb/include/lldb/Symbol/Function.h:453 /// PC addresses (in increasing order). - llvm::MutableArrayRef<CallEdge> GetCallEdges(); + llvm::ArrayRef<std::unique_ptr<CallEdge>> GetCallEdges(); ---------------- Since DWARFExpression has a copy constructer, perhaps we should get rid of the unique_ptr to make it faster to allocate/deconstruct? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70100/new/ https://reviews.llvm.org/D70100 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
