[Lldb-commits] [PATCH] D80793: [lldb][NFC] Make ClangExpressionSourceCode's wrapping logic more consistent
Shafik Yaghmour via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Jun 1 18:58:06 PDT 2020
shafik added inline comments.
Comment at: lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionSourceCode.h:33-40
+ /// Wrapped in a member function of a C++ class.
+ /// Wrapped in a instance Objective-C method.
+ /// Wrapped in a static Objective-C method.
+ /// Wrapped in a non-member function.
> If I understand the code correctly, a c++ static member function is wrapped using the "function" approach. I think this is potentially confusing, so it would be good if the comments elaborated on it more (e.g. add "non-static" to the CppMemberFunction description, and explicitly mention static member functions in the "function" description).
It might be worth adding that we don't need to find the `this` for the C++ case unlike a non-static member function. This happens in `LookUpLldbClass`
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits