[llvm] [llvm][DebugInfo] Encode DW_AT_object_pointer on method declarations with DW_FORM_implicit_const (PR #124790)

[Michael Buch via llvm-commits]

Michael137 wrote:

> > @labath - any thoughts on this?
> 
> Maybe?
> 
> * I am wondering how other debuggers (gdb mainly) will react to the new attribute. It would be great if "tuning" for lldb didn't completely break other debuggers as well

Good point, will play around with gdb to see if this breaks them when tuning for lldb

> * speaking of tuning, I believe we currently don't "tune" Google binaries for lldb, so if this is conditioned on -glldb and we still emit the long attribute for gdb, then this will not change the size difference you were seeing.

The current patch would basically not emit the attribute anymore for non-lldb tuned binaries (which we weren't doing anyway up until very recently). So the size regression would be mitigated there also

> We probably should start tuning for lldb, but I don't know what will all of that cause (in particular it could cause a size _increase_ due to some extra attributes emitted this way, but I don't think the difference should be big). I don't think this is Michael's problem, I'm just saying...

Yea would be an interesting experiment


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