Re: basic-improvements merge status

[Andrew Pinski <apinski at apple dot com>]

This is the same problem on Darwin but it shows up sooner and in the 
build of libmath.

Thanks,
Andrew Pinski

On Monday, Dec 16, 2002, at 14:38 US/Pacific, David Edelsohn wrote:

> > > > > > Jan Hubicka writes:
> > > #ifndef HAVE_SINF
> > > float
> > > sinf(float x)
> > > {
> > > return (float) sin(x);
> > > }
> > > #endif
> > >
> > > is being compiled as
> Jan> I think we can deal with -fno-builtin-fsin to avoid gcc from being
> Jan> active on optimizing this.
> Jan> The name sinf is reserved by C90 standard, so this is not valid C.
>
> float
> foo (float x)
> {
>   return (float) sin(x);
> }
>
> always should be valid without the user having to specify 
> -fno-builtin, as
> your change requires on systems without sinf().  stubs.c could be 
> compiled
> with -fno-builtin, but that's just covering up the incorrect 
> assumption in
> the transformation.
>
> David