<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jul 28, 2016 at 2:14 PM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">grimar added inline comments.<br>
<span class=""><br>
================<br>
Comment at: ELF/LinkerScript.cpp:922-923<br>
@@ -918,3 +921,4 @@<br>
   if (Tok.getAsInteger(0, V)) {<br>
-    if (!isValidCIdentifier(Tok))<br>
+    if (Tok != "." && !isValidCIdentifier(Tok))<br>
       setError("malformed number: " + Tok);<br>
+    return [=](uint64_t Dot) { return getSymbolValue(Tok, Dot); };<br>
----------------<br>
</span><span class="">ruiu wrote:<br>
> I think you can remove this error check. If Tok is not a valid symbol name, getSymbolValue will find it.<br>
</span>Looks few tests begins to fail if I remove. Since this line is not added by this patch, I suggest to leave it for now, I'll check how to remove it and prepare a patch tomorrow.<br></blockquote><div><br></div><div>Sounds good.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="https://reviews.llvm.org/D22916" rel="noreferrer" target="_blank">https://reviews.llvm.org/D22916</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>