<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Aug 2, 2016 at 3:27 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 2 August 2016 at 18:03, Davide Italiano <<a href="mailto:davide@freebsd.org">davide@freebsd.org</a>> wrote:<br>
> On Tue, Aug 2, 2016 at 5:59 AM, Rafael Espíndola<br>
> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>>> +  next();<br>
>>> +  if (Tok.startswith("=0x"))<br>
>>> +    return parseHex(Tok.substr(3));<br>
>><br>
>> This is missing the error check for Value below.<br>
>><br>
><br>
> My understanding is that we support arbitrary hex values, the<br>
> restriction is only for decimal.<br>
<br>
</span>Why?</blockquote><div><br></div><div>In this context 0xbeef is different from 0x0000beef. The former is 2 byte padding while the latter is 4 bytes.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
>>> +  // This must be a decimal.<br>
>>> +  unsigned int Value;<br>
>>> +  if (Tok.substr(1).getAsInteger(10, Value)) {<br>
>>> +    setError("filler should be a decimal/hexadecimal value");<br>
>>>      return {};<br>
>>>    }<br>
>>> -  Tok = Tok.substr(3);<br>
>>> -  next();<br>
>>> -  return parseHex(Tok);<br>
>>> +  if (Value > 255)<br>
>>> +    setError("only single bytes decimal are supported for the filler now");<br>
>><br>
>> I think you can just pass a base of 0 to getAsInteger and it will<br>
>> detect the base.<br>
>><br>
><br>
> I'm fine with this, but at this point we can also accept octals, and<br>
> I'm not sure we want to do that. Does this make sense?<br>
<br>
</span>A quick check shows that both bfd and gold do support octal. Since<br>
that actually makes the code simpler, I would be for it.<br>
<br>
Thanks,<br>
Rafael<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">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>
</div></div></blockquote></div><br></div></div>