[PATCH] D49653: [LLD] [COFF] Handle comdat sections without leader symbols
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 23 22:07:18 PDT 2018
mstorsjo added inline comments.
================
Comment at: COFF/InputFiles.cpp:289
readAssociativeDefinition(Sym, Def);
Symbols[I] = createRegular(Sym);
}
----------------
rnk wrote:
> I think this is probably the right place to do the check.
Yes, this is the one place where `createRegular` is called without checking for `PendingComdat` before. I thought the check felt natural there, but I can move it here instead.
Repository:
rLLD LLVM Linker
https://reviews.llvm.org/D49653
More information about the llvm-commits
mailing list