<div dir="ltr">No worries. This patch has already been rolled back.<div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 11, 2016 at 1:04 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Gah, this basically reverted my patch.<br>
<div><div><br>
On 11 March 2016 at 13:25, Rui Ueyama <<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>> wrote:<br>
> ruiu added inline comments.<br>
><br>
> ================<br>
> Comment at: lld/trunk/ELF/InputFiles.cpp:439-465<br>
> @@ -482,16 +438,29 @@<br>
><br>
>  void BitcodeFile::parse(DenseSet<StringRef> &ComdatGroups) {<br>
>    LLVMContext Context;<br>
>    std::unique_ptr<IRObjectFile> Obj = check(IRObjectFile::create(MB, Context));<br>
>    const Module &M = Obj->getModule();<br>
><br>
>    DenseSet<const Comdat *> KeptComdats;<br>
>    for (const auto &P : M.getComdatSymbolTable()) {<br>
>      StringRef N = Saver.save(P.first());<br>
>      if (ComdatGroups.insert(N).second)<br>
>        KeptComdats.insert(&P.second);<br>
>    }<br>
><br>
> -  for (const BasicSymbolRef &Sym : Obj->symbols())<br>
> -    if (!shouldSkip(Sym))<br>
> -      SymbolBodies.push_back(createSymbolBody(KeptComdats, *Obj, Sym));<br>
> +  for (const BasicSymbolRef &Sym : Obj->symbols()) {<br>
> +    const GlobalValue *GV = Obj->getSymbolGV(Sym.getRawDataRefImpl());<br>
> +    assert(GV);<br>
> +    uint32_t Flags = Sym.getFlags();<br>
> +    if (const Comdat *C = GV->getComdat())<br>
> +      if (!KeptComdats.count(C))<br>
> +        continue;<br>
> +    if (!(Flags & BasicSymbolRef::SF_Global))<br>
> +        continue;<br>
> +    if (GV->hasAppendingLinkage()) {<br>
> +      ExtraKeeps.push_back(GV->getName().copy(Alloc));<br>
> +      continue;<br>
> +    }<br>
> +    if (Flags & BasicSymbolRef::SF_FormatSpecific)<br>
> +      continue;<br>
> +    uint8_t Visibility = getGvVisibility(GV);<br>
> ----------------<br>
> Isn't this an accidental change?<br>
><br>
><br>
> Repository:<br>
>   rL LLVM<br>
><br>
> <a href="http://reviews.llvm.org/D18085" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18085</a><br>
><br>
><br>
><br>
</div></div></blockquote></div><br></div></div>