[PATCH] D47391: [COFF] Simplify symbol table output section computation
Shoaib Meenai via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 29 14:56:14 PDT 2018
smeenai added inline comments.
================
Comment at: COFF/Writer.cpp:638
switch (Def->kind()) {
case Symbol::DefinedAbsoluteKind:
----------------
smeenai wrote:
> pcc wrote:
> > Can we move this switch to the top of the function? Then the only code that needs to care about output sections will be in the default case of the switch.
> Good idea; I'll throw up a patch in a bit.
https://reviews.llvm.org/D47497
Repository:
rLLD LLVM Linker
https://reviews.llvm.org/D47391
More information about the llvm-commits
mailing list