[PATCH] D78954: [LLD][ELF] Add isDebug flag to SectionBase.
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 27 13:28:27 PDT 2020
MaskRay added a comment.
Most links don't use --strip-debug or --strip-all, so `isDebugSection` in `LinkerDriver::link` is rarely called.
Most links don't specify -r or --emit-relocs, so `InputSection::copyRelocations` is not called. In addition, `isDebugInfo` inside the function is called by edge cases which we have to suppress warnings for practical reasons, the cases are even fewer.
I am hesitant to accept this patch, unless you can make further justification.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78954/new/
https://reviews.llvm.org/D78954
More information about the llvm-commits
mailing list