2013/11/27 Jeff Law <l...@redhat.com>: > On 11/26/13 19:50, Jan-Benedict Glaw wrote: >> >> On Tue, 2013-11-26 04:26:57 +0100, Jan-Benedict Glaw <jbg...@lug-owl.de> >> wrote: >>> >>> Build log at >>> http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=39052 >>> >>> g++ -c -DUSE_LIBUNWIND_EXCEPTIONS -g -O2 -DIN_GCC >>> -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions -fno-rtti >>> -fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwrite-strings >>> -Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-long >>> -Wno-variadic-macros -Wno-overlength-strings -Werror -fno-common >>> -DHAVE_CONFIG_H -I. -I. -I../../../gcc/gcc -I../../../gcc/gcc/. >>> -I../../../gcc/gcc/../include -I../../../gcc/gcc/../libcpp/include >>> -I/opt/cfarm/mpc/include -I../../../gcc/gcc/../libdecnumber >>> -I../../../gcc/gcc/../libdecnumber/dpd -I../libdecnumber >>> -I../../../gcc/gcc/../libbacktrace -o ia64.o -MT ia64.o -MMD -MP -MF >>> ./.deps/ia64.TPo ../../../gcc/gcc/config/ia64/ia64.c >>> In file included from ./tm.h:20:0, >>> from ../../../gcc/gcc/config/ia64/ia64.c:25: >>> ../../../gcc/gcc/config/ia64/hpux.h:185:34: error: >>> ‘default_c99_libc_has_function’ was not declared in this scope >>> #define TARGET_LIBC_HAS_FUNCTION default_c99_libc_has_function >>> ^ >>> ./target-hooks-def.h:1140:5: note: in expansion of macro >>> ‘TARGET_LIBC_HAS_FUNCTION’ >>> TARGET_LIBC_HAS_FUNCTION, \ >>> ^ >>> ../../../gcc/gcc/config/ia64/ia64.c:659:29: note: in expansion of macro >>> ‘TARGET_INITIALIZER’ >>> struct gcc_target targetm = TARGET_INITIALIZER; >>> ^ >>> make[2]: *** [ia64.o] Error 1 >> >> >> I *guess* it should have been no_c99_libc_has_function instead of >> default_c99_libc_has_function, referring to an equal change to >> pa/hpux.h in r201838: >> >> http://gcc.gnu.org/viewcvs/gcc?view=revision&revision=201838 >> >> http://gcc.gnu.org/git/?p=gcc.git;a=commit;h=30f690e026ecdf99c68e777a48562b58afe37f43 >> >> But I don't actually know HP-UX's libc and whether or not it ships an >> equal set of functions on all target it runs on...
"no_c99_libc_has_function" would be incorrect: before we switched to target hook implementation of checking the presence of builtin functions, we had for hpux: /* ia64 HPUX has the float and long double forms of math functions. */ #undef TARGET_C99_FUNCTIONS #define TARGET_C99_FUNCTIONS 1 And since now we assume that by default c99 functions are present, the correct version of the hook would be exactly the default one. > I approved a patch today that I think will fix this. > Jeff That's right, it is committed: http://gcc.gnu.org/ml/gcc-cvs/2013-11/msg01158.html thanks, Alexander
