Enable non-complex math builtins from C99 for Bionic

Hi, there are still a couple of problems with my patch:

The build is broken for the following targets:
1) *linux* targets that do not include config/linux.h in their tm.h
(e.g alpha-linux, ppc64-linux etc). For them we have:

../../../../gcc/gcc/config/linux-android.c: In function ‘bool
linux_android_libc_has_function(function_class)’:
../../../../gcc/gcc/config/linux-android.c:40:7: error:
‘OPTION_BIONIC’ was not declared in this scope
   if (OPTION_BIONIC)
       ^
make[2]: *** [linux-android.o] Error 1

This is adressed in the changes of config/linux-android.c: linux_libc,
LIBC_GLIBC and LIBC_BIONIC seem to be declared for all *linux*
targets.

2) *uclinux* targets that include config/linux.h. For *uclinux* we do
not use linux-protos.h, and therefore linux_android_libc_has_function
is not declared there.
I don't want to add aditional tmake_file, tm_p_file and extra_objs, so
I added explicit define of TARGET_LIBC_HAS_FUNCTION as
no_c99_libc_has_function for those targets.

I'm sorry for that. The following patch cured my build of those
targets; it is also preserving the initial presence of c99. There were
plenty of targets that were changed by my patch, I hope this time I
didn't miss anything.

Is it ok?

+#define TARGET_LIBC_HAS_FUNCTION no_c99_libc_has_function

thanks,
Alexander

2013/8/21 Rainer Orth <ro@cebitec.uni-bielefeld.de>:
> Alexander Ivchenko <aivchenk@gmail.com> writes:
>
>> Hi Joseph, thanks for your comments.
>>
>> I updated the patch:
>>
>> 1) The function name as a second argument in libc_has_function target
>> hook was removed - was not usefull so far.
>> 2) By using contrib/config-list.mk (thanks for the hint - great tool!)
>> and analysing tm.h files and what is included in them I have checked
>> 197 targets. That analysis includes all issues that you raised in your
>> comments - everything is fixed now. I don't like that sometimes we
>> have to redefine the version of the hook back to the default one due
>> to a poisoning of including elfos.h, but I couldn't find a better
>> solution - I commented all those cases.
>>
>> Regtesting is in progress now. I have already tested the patch before,
>> so I don't expect to see any new problems.
>>
>> If all the tests pass, is the patch OK for trunk?
>
> Unfortunately, this patch broke Solaris 10+ bootstrap; it cannot have
> been tested properly there:
>
> In file included from ./tm.h:27:0,
>                  from /vol/gcc/src/hg/trunk/local/gcc/gencheck.c:23:
> /vol/gcc/src/hg/trunk/local/gcc/config/sol2-10.h:21:4: error: "/*" within comment [-Werror=comment]
>  /* /* Solaris 10 has the float and long double forms of math functions.
>  ^
> cc1plus: all warnings being treated as errors
> make[3]: *** [build/gencheck.o] Error 1
>
> Fixed as follows; bootstrapped without regressions on
> i386-pc-solaris2.10, installed on mainline.
>
>         Rainer
>
>
> 2013-08-21  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
>
>         * config/sol2-10.h (TARGET_LIBC_HAS_FUNCTION): Don't nest
>         comment.
>
>
>
> --
> -----------------------------------------------------------------------------
> Rainer Orth, Center for Biotechnology, Bielefeld University
>

On Wed, Aug 21, 2013 at 11:21:32PM +0400, Alexander Ivchenko wrote:
> I'm sorry for that. The following patch cured my build of those
> targets; it is also preserving the initial presence of c99. There were
> plenty of targets that were changed by my patch, I hope this time I
> didn't miss anything.

S/390 bootstrap still fails. The reason is that when we set
tm_file/tm_p_file in config.gcc we do not append to the existing
content so your adjustments done before are ignored and gcc complains
about unknown linux_android_libc_has_function.

However, since with s390 Linux and TPF we only target GNU Linux
systems with Glibc we can default to gnu_libc_has_functions anyway:

Index: gcc/config/s390/linux.h
===================================================================
*** gcc/config/s390/linux.h.orig        2013-01-14 07:48:06.000000000 +0000
--- gcc/config/s390/linux.h     2013-08-22 07:57:46.006014197 +0000
*************** along with GCC; see the file COPYING3.
*** 87,90 ****
--- 87,93 ----
  /* Define if long doubles should be mangled as 'g'.  */
  #define TARGET_ALTERNATE_LONG_DOUBLE_MANGLING
  
+ #undef TARGET_LIBC_HAS_FUNCTION
+ #define TARGET_LIBC_HAS_FUNCTION gnu_libc_has_function
+ 
  #endif
Index: gcc/config/s390/tpf.h
===================================================================
*** gcc/config/s390/tpf.h.orig  2013-08-22 07:01:48.000000000 +0000
--- gcc/config/s390/tpf.h       2013-08-22 07:57:27.706013534 +0000
*************** along with GCC; see the file COPYING3.
*** 111,118 ****
  /* IBM copies these libraries over with these names.  */
  #define MATH_LIBRARY "CLBM"
  #define LIBSTDCXX "CPP2"
- #endif /* ! _TPF_H */
  
- /* We redefine this hook so the version from elfos.h header won't be used.  */
  #undef TARGET_LIBC_HAS_FUNCTION
! #define TARGET_LIBC_HAS_FUNCTION default_libc_has_function
--- 111,118 ----
  /* IBM copies these libraries over with these names.  */
  #define MATH_LIBRARY "CLBM"
  #define LIBSTDCXX "CPP2"
  
  #undef TARGET_LIBC_HAS_FUNCTION
! #define TARGET_LIBC_HAS_FUNCTION gnu_libc_has_function
! 
! #endif /* ! _TPF_H */