Re: __builtin_mul_overflow_p support with llvm 5.0

2017-10-30 Thread Paul Eggert
Paul Eggert wrote: Sorry, what's the context here? Ah, I see it now, here: https://github.com/coreutils/coreutils/issues/11 where Bernhard Rosenkränzer wrote: "Our clang package is (intentionally) patched to return GNUC == 7 because clang 5.0 is closer to gcc 7.x than to any other version o

Re: __builtin_mul_overflow_p support with llvm 5.0

2017-10-30 Thread Paul Eggert
Sorry, what's the context here? In the past we've avoided having intprops.h depend on configure-time checks and to some extent I suppose I'd rather continue in that tradition, if only because that'll make it easier to fold into glibc one of these days. Can you reformulate the patch so that it

__builtin_mul_overflow_p support with llvm 5.0

2017-10-30 Thread Pádraig Brady
Forwarding patch to gnulib where it would be adjusted/applied. cheers, Pádraig --- coreutils-8.28/configure.ac.omv~2017-10-30 17:14:36.543445080 +0100 +++ coreutils-8.28/configure.ac 2017-10-30 17:27:19.273190455 +0100 @@ -330,6 +330,18 @@ if test $ac_cv_func_syslog = no; then done fi +A