<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Dec 21, 2016 at 9:12 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">Rui Ueyama via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> writes:<br>
<br>
<br>
> Log:<br>
> De-template DefinedSynthetic.<br>
<br>
Thanks!<br>
<span class=""><br>
>      if (!Sec)<br>
>        return D.Value;<br>
> -    if (D.Value == DefinedSynthetic<ELFT>::<wbr>SectionEnd)<br>
> +    if (D.Value == uintX_t(-1))<br>
<br>
</span>Having a name for the -1 was probably a good thing.<br></blockquote><div><br></div><div>I changed the type of `Value` to uint64_t, so I had to change `SectionEnd` type to uint64_t. But if I did, compiler warns on a type conversion from a uint64_t constant -1 to uintX_t if uintX_t is a 32-bit type. That's why I chose to remove the constant.</div></div></div></div>