On Fri, 21 Dec 2012, Alexander Ivchenko wrote:

> Hi,
> 
> Thank you very much for your input! Please, take a look at the updated 
> version:
> I fixed coding style, moved documentation for TARGET_LIBC_HAS_FUNCTION
> to target.def.
> Removed TARGET_C99_FUNCTIONS and TARGET_HAS_SINCOS and all their
> influence and moved the implementation of linux_libc_has_function to
> host-linux.c.
>       I changed the defaults: now it is assumed that we have C99 runtime,
> but no sincos. I updated all needed gcc/config/*.h. But 'm not sure in
> this part,
> cause I don't have the opportunity to test it properly...

This patch seems mostly plausible, though there are various places that 
call targetm.libc_has_function with and empty string as second argument, 
that should be naming the specific function instead.  I haven't reviewed 
the details, and at this development stage I think it will need to wait 
until after 4.8 branches.

-- 
Joseph S. Myers
jos...@codesourcery.com

Reply via email to