Patch for missing sinf, cosf, ...

Steven King sxking@uswest.net
Wed Jun 7 11:00:00 GMT 2000


On Wed, 07 Jun 2000, Branko Èibej wrote:
> 
> Benjamin Kosnik wrote:
> > 
> > I just checked in the long double patch from Steven King, which stomps on
> > this stuff (HAVE_BUILTIN, etc). Can you update this for Gaby?
> 
> O.K., attached. The ChangeLog entry doesn't change.
> 
> BTW, if these patches are O.K., would it make sense to do the same for
> the other *f and *l built-ins that are found?

While I think the basic thrust here is correct, I dont believe the replacement
functions are neccessary;  I think it should be sufficient instead to just tweek
bits/std_cmath.h and shadow/bits/std_cmath.h.

s/BUILTIN_\([A-Z0-9]\+\)/s//BUILTIN_\1 \&\& _GLIBCPP_HAVE_\1/g

or perhaps

s/BUILTIN_\([A-Z0-9]\+\)/s//BUILTIN_\1 \&\& (_GLIBCPP_HAVE_\1 || _GLIBCPP_HAVE__\1)/g

this would have the advantage of inlining the call to the replacement (and
would handle all the other *f and *l that are found).

  -- 
Steven King
sxking@uswest.net


More information about the Libstdc++ mailing list