[PATCH] D40845: [WebAssembly] COMDAT: LLD support
Sam Clegg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 10 11:52:05 PST 2018
sbc100 added inline comments.
================
Comment at: wasm/InputFiles.cpp:271
+ if (!Comdat.empty() && Symtab->findComdat(Comdat) != this)
+ Seg->Discarded = true;
+ }
----------------
Can we check the comdats when the segment or function is first created to avoid two loops?
Maybe we could then avoid even adding the to the Functions or Segments list at all and just discard them?
Repository:
rLLD LLVM Linker
https://reviews.llvm.org/D40845
More information about the llvm-commits
mailing list