<p dir="ltr"><br>
On Mar 24, 2016 4:10 PM, "Davide Italiano" <<a href="mailto:dccitaliano@gmail.com">dccitaliano@gmail.com</a>> wrote:<br>
><br>
> davide marked 4 inline comments as done.<br>
> davide added a comment.<br>
><br>
> Oh, the testcase was there before (just didn't show up in my last rev, weird).<br>
> Addressing some of your comments + reply.<br>
><br>
><br>
> ================<br>
> Comment at: ELF/LTO.cpp:113<br>
> @@ +112,3 @@<br>
> +          // symbols when creating a shared library.<br>
> +          if (!Config->Shared && !Used.count(GV) && !B->isUsedInRegularObj())<br>
> +            InternalizedSyms.insert(GV->getName());<br>
> ----------------<br>
> rafael wrote:<br>
> > ExportDynamic should have the same treatment as shared. Not sure if we already have a convenient predicate for either.<br>
> Can I do this separately? (And add a specific test for it)</p>
<p dir="ltr">Yes, good point.</p>
<p dir="ltr">> ================<br>
> Comment at: ELF/SymbolTable.cpp:221<br>
> @@ +220,3 @@<br>
> +  if (auto *BC = dyn_cast<DefinedBitcode>(Existing))<br>
> +    if (auto *U = dyn_cast<Undefined>(New))<br>
> +      BC->setUsedInRegularObj();<br>
> ----------------<br>
> rafael wrote:<br>
> > I am not sure just Undefined is sufficient.<br>
> ><br>
> > If a weak symbol in an object file is replaced with a strong symbol in a .bc file, there can be a reference from the strong symbol to the bit code one.<br>
> ><br>
> Can I do this separately?</p>
<p dir="ltr">This one I think is simpler if you do it now. Basically it is not important if the symbol is undefined or not. Just that it is from an object file.<br></p>
<p dir="ltr">><br>
><br>
><a href="http://reviews.llvm.org/D18415"> http://reviews.llvm.org/D18415</a><br>
><br>
><br>
><br>
</p>