teemperor added a comment.

This LGTM, but the TODO directly above the change in ClangExpressionDeclMap.cpp 
worries me a bit. I am not sure how good our test coverage is for calling 
functions when there are instance methods in scope (e.g., we are in instance 
method and try to call another instance method).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70846



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

Reply via email to