[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