<div dir="ltr">That's right. I just wrote this patch, but have a bad short memory, ha!</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 30, 2017 at 12:20 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">On 30 March 2017 at 15:18, Rui Ueyama via Phabricator<br>
<span class=""><<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br>
> ruiu added inline comments.<br>
><br>
><br>
> ================<br>
> Comment at: lld/ELF/Relocations.cpp:623<br>
> +  std::string Obj = S.getObjMsg<ELFT>(Offset);<br>
> +  if (Src.empty() && !Obj.empty())<br>
> +    Msg += "\n>>> referenced by " + Obj;<br>
> ----------------<br>
> rafael wrote:<br>
>> Obj is never empty, is it?<br>
> If it is a synthetic section, it can be null, I think.<br>
<br>
</span>It returns  "(internal)" for those.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>