[lld] r290241 - De-template DefinedSynthetic.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 21 20:26:13 PST 2016


On Wed, Dec 21, 2016 at 9:12 AM, Rafael Avila de Espindola <
rafael.espindola at gmail.com> wrote:

> Rui Ueyama via llvm-commits <llvm-commits at lists.llvm.org> writes:
>
>
> > Log:
> > De-template DefinedSynthetic.
>
> Thanks!
>
> >      if (!Sec)
> >        return D.Value;
> > -    if (D.Value == DefinedSynthetic<ELFT>::SectionEnd)
> > +    if (D.Value == uintX_t(-1))
>
> Having a name for the -1 was probably a good thing.
>

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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161221/170a2075/attachment.html>


More information about the llvm-commits mailing list