shafik marked an inline comment as done.
shafik added a comment.

In D69913#1739579 <https://reviews.llvm.org/D69913#1739579>, @jingham wrote:

> Why not have the FindFunctions lambda take a FunctionSP?  It would be easy to 
> get the Function name out of the function in the lambda, and that would make 
> the function more general (and also match what the Foreach does...


It was not obvious how they were using it at first, good catch.


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

https://reviews.llvm.org/D69913



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to