Re: [Tinycc-devel] Annoying new warning

[Christian JULLIEN]

Hi Vincent,

return 666; works equally well as it should not be called by ARM backend.

This is a workaround to remove warning.

If in C++, I would raise an exception instead.

>  Le : 05 mai 2017 à 08:56 (GMT +02:00)
> De : "Vincent Lefevre" <address@hidden>
> À : "address@hidden" <address@hidden>
> Objet : Re: [Tinycc-devel] Annoying new warning
>
>
> On 2017-05-05 06:46:27 +0200, Christian Jullien wrote:
> > Ok, I tried this patch which allowed me to run my complete non-regression
> > test suite without problem on ARM.
> > 
> > Do you agree I push this patch in mob?
> > 
> > diff --git a/lib/libtcc1.c b/lib/libtcc1.c
> > index 9195489..bcdfb0b 100644
> > --- a/lib/libtcc1.c
> > +++ b/lib/libtcc1.c
> > @@ -577,6 +577,10 @@ unsigned long long __fixunsdfdi (double a1)
> > 
> >  unsigned long long __fixunsxfdi (long double a1)
> >  {
> > +#if defined(TCC_TARGET_ARM)
> > +//    (void)fprintf(stderr, "__fixunsxfdi(%g) called for TCC_TARGET_ARM\n",
> > a1);
> > +    return 0;
>
> Why should the result be 0 on ARM?
>
> -- 
> Vincent Lefèvre <address@hidden> - Web: <https://www.vinc17.net/>
> 100% accessible validated (X)HTML - Blog: <https://www.vinc17.net/blog/>
> Work: CR INRIA - computer arithmetic / AriC project (LIP, ENS-Lyon)
>
> _______________________________________________
> Tinycc-devel mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/tinycc-devel