[Lldb-commits] [lldb] [LLDB] Add field member operators to DIL (PR #138093)

via lldb-commits lldb-commits at lists.llvm.org
Wed May 7 23:13:56 PDT 2025


cmtice wrote:

> A very predictable question. Why is the implementation of this more complicated than `base->GetChildMemberWithName(name)`. I think it shouldn't be.
> 
> To avoid repeating the same points over and over, I suggest going through the discussion on the dereference/addressof patch. If you still have questions after that, let me know.

GetChildMemberWithName didn't handle everything correctly. I've fixed part of that now. There's still a bug in the code that searches for fields in base classes, which I want to try to find & fix.  I've removed all the special handling code from this PR.

Please review again. Thanks!

https://github.com/llvm/llvm-project/pull/138093


More information about the lldb-commits mailing list