[llvm] [objcopy][coff] Place section name first in strtab (PR #145266)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 23 00:40:57 PDT 2025


https://github.com/jh7370 approved this pull request.

This looks reasonable to me. I guess there are extreme cases (e.g. ridiculously long section names/way too many sections) where the original issue could still be exposed, but that feels outside the responsibility of llvm-objcopy to worry about.

LGTM, but I'd like someone with more COFF knowledge, e.g. @mstorsjo or @MaskRay to review too.

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


More information about the llvm-commits mailing list