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

Alexander Yermolovich via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 7 17:56:42 PST 2023


ayermolo wrote:

Thank you for putting up a PR. Looking at it I am not sure what the fundamental differences are between your patch and mine. It looks like it is somewhat where my patch has started where additional relocation checks remained, and consequently removed per reviewers suggestions. I thought my PR was going in the right direction per your comment https://github.com/llvm/llvm-project/pull/70701#pullrequestreview-1728476884
Can you explain why the direction you chose to go with is better?

Thanks.

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


More information about the llvm-commits mailing list