<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Oct 21, 2015 at 4:13 PM, Davide Italiano <span dir="ltr"><<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Wed, Oct 21, 2015 at 3:53 PM, Rui Ueyama via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
> ruiu added a comment.<br>
><br>
> Oh yeah, that's very unfortunate, but that's true.<br>
><br>
> I took a look at gold's source code and found that their lexer behaves differently depending on the context -- if it's outside of version script, ':' is a token character, but inside version script, only '::' are handled as part of a token. I don't know if we really want to have this craziness.<br>
><br>
<br>
</span>Yes, I noticed that as well and I was a little bit scared by the complexity.<br>
<span><br>
> I'd really like to always handle ':' as a token character, and let users to update their linker script if not compatible with that behavior. Davide, could you grep all linker scripts in FreeBSD which has a space between 'global' or 'local' and the following semicolon and count that number?<br>
><br>
<br>
</span>Me too. FWIW, a quick scan in FreeBSD showed up none of the version<br>
scripts has the space. Even if it would have, it doesn't matter, given<br>
I have the power to change it. I can't speak for Linux, other OSes and<br>
all the majority of people using version scripts. In other words, I<br>
just tried to be a "good citizen" and look outside of FreeBSD. If you,<br>
Rafael et al. don't have objections to support semicolons as token<br>
character I'll just go for it (and we can re-evaluate the decision<br>
later, if needed).<br></blockquote><div><br></div><div>I have no objection -- I do support handling ':' just as a regular token character in order to avoid complex tokenization logic.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
><br>
> <a href="http://reviews.llvm.org/D13960" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13960</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<span><font color="#888888"><br>
--<br>
Davide<br>
<br>
"There are no solved problems; there are only problems that are more<br>
or less solved" -- Henri Poincare<br>
</font></span></blockquote></div><br></div></div>