On Tue, Oct 18, 2022 at 11:39:25AM +0200, Florian Weimer via Gcc-patches wrote:
> libsanitizer/
>
> * configure.ac (check for necessary platform features):
I'd use (sanitizer_supported) or (SANITIZER_SUPPORTED) above instead,
that is what is what is being determined by the test.
> Include <unistd.h> for syscall prototype.
> * configure: Regenerate.
Otherwise LGTM, thanks.
> --- a/libsanitizer/configure.ac
> +++ b/libsanitizer/configure.ac
> @@ -161,7 +161,8 @@ case "$target" in
> *-*-linux*)
> # Some old Linux distributions miss required syscalls.
> sanitizer_supported=no
> - AC_TRY_COMPILE([#include <sys/syscall.h>],[
> + AC_TRY_COMPILE([#include <sys/syscall.h>
> +#include <unistd.h>],[
> syscall (__NR_gettid);
> syscall (__NR_futex);
> syscall (__NR_exit_group);
>
> base-commit: acdb24166d13d87c374e578d2ad5d58249171930
Jakub
