[lld] [ELF] Change .debug_names tombstone value to UINT32_MAX/UINT64_MAX (PR #74686)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 21 18:58:00 PST 2023


MaskRay wrote:

> Thank you for explanation. I think it would be valuable for me, and for the health of the project, if next time we could continue discussion in the original patch I think it would benefit everyone. :)

Thank you for the review and your input. Quality and cohesion are paramount for the project's success.
I consider how we can balance both aspects moving forward to ensure a smoother collaborative process while upholding the project's standards.
I sometimes suggest the description but sometimes it might be better to have a different version.

> Do we need 64 bit test?

This version includes both 64-bit and 32-bit tests:)

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


More information about the llvm-commits mailing list