[PATCH] D127148: [ELF] Support 'G' in lld
Florian Mayer via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 6 18:06:32 PDT 2022
fmayer added a comment.
In D127148#3562065 <https://reviews.llvm.org/D127148#3562065>, @MaskRay wrote:
> llvm/lib/DebugInfo/DWARF/DWARFDebugFrame.cpp code can be tested with llvm-objdump/llvm-dwarfdump but should have a test in `test/DebugInfo`.
>
> lld doesn't use llvm/lib/DebugInfo/DWARF/DWARFDebugFrame.cpp code. It's inappropriate to rely on lld/test/ELF to provide the test coverage.
>
> You should move llvm/lib/DebugInfo/DWARF/DWARFDebugFrame.cpp into another patch with a test/DebugInfo patch.
Done, moved DWARF change to D127171 <https://reviews.llvm.org/D127171>.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127148/new/
https://reviews.llvm.org/D127148
More information about the llvm-commits
mailing list