[PATCH] D136395: Add the ability to verify the .debug_aranges section.
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 20 18:06:46 PDT 2022
dblaikie added a comment.
If possible, I'd rather not add this - I think .debug_aranges should be removed (it's already been off-by-default for a decade in Clang) in favor of using CU-level address ranges. They're cheap-enough to parse that it doesn't substantially change the performance of tools so far as I'm aware and they save space by not duplicating the address range information in two places.
Adding a verifier feels like endorsing/encouraging/maintaining `.debug_aranges` which seems like the wrong direction we should be going.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D136395/new/
https://reviews.llvm.org/D136395
More information about the llvm-commits
mailing list