[PATCH] D107470: [llvm+lldb] Remove dead-code in DWARFListTableHeader::extract modifying DWARFDataExtractor
Jan Kratochvil via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 16 11:24:37 PDT 2021
jankratochvil marked 3 inline comments as done.
jankratochvil added a comment.
In D107470#2946689 <https://reviews.llvm.org/D107470#2946689>, @ikudrin wrote:
> `DWARFDebugInfo.TestRnglistsAddressSize`, `DWARFListTableHeader.AddressSize64Offset`, and `DWARFListTableHeader.AddressSize32Offset` pass without applying the patch. Why adding them?
This code has shown it is prone to errors so future changes could break some other combination. Better code coverage by the testsuite is AFAIK always preferred.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107470/new/
https://reviews.llvm.org/D107470
More information about the llvm-commits
mailing list