<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jan 11, 2017 at 11:45 AM, Rafael Avila de Espindola <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"><span class="">Rui Ueyama via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> writes:<br>
<br>
> ruiu added inline comments.<br>
><br>
><br>
> ================<br>
> Comment at: ELF/InputSection.cpp:38-41<br>
> +  // File can absent if section is synthetic.<br>
> +  if (!Sec->getFile())<br>
> +    return ("'" + Sec->Name + "'").str();<br>
>    return (Sec->getFile()->getName() + ":(" + Sec->Name + ")").str();<br>
> ----------------<br>
> If a section is a synthetic section, it's output will be something like<br>
><br>
>   'foo'<br>
><br>
> while if it is not, it will be something like<br>
><br>
>   bar.o:(foo)<br>
><br>
> So, apparently they are inconsistent in quoting style. That's not good.<br>
<br>
</span>Would you prefer<br>
<br>
(foo)<br>
<br>
and<br>
bar.o:(foo)<br><br></blockquote><div>I think so. </div></div><br></div></div>