firmiana402 wrote:

> I think it would be worth calling out the deliberate compat trade-off: 
> pre-existing, DWARF<4, and SCE-tuned 32-bit binaries with wide DW_OP_const* 
> are now truncated by the consumer.

Thanks for the review. I've updated the commit to address code style and test 
issues.

Regarding the compatibility trade-off you mentioned – I agree it's worth 
calling out. In my view, it's unavoidable here, and the practical impact is 
likely limited. On the flip side, this fix actually improves compatibility with 
other debuggers like GDB, because LLVM's producer has historically been 
inconsistent with them. Also, DWARF v4 has been available since 2010, and 
fixing both sides together helps break the long‑standing deadlock between 
producers and consumers.

https://github.com/llvm/llvm-project/pull/204353
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to