[PATCH][GCC 10] Fix build failure with zstd versio9n 1.2.0 or older.

Richard Biener richard.guenther@gmail.com
Wed Sep 30 06:41:02 GMT 2020


On Wed, Sep 30, 2020 at 5:56 AM Jim Wilson <jimw@sifive.com> wrote:
>
> This is the gcc-10 branch version of the patch on mainline.
>
> Extends the configure check for zstd.h to also verify the zstd version,
> since gcc requires features that only exist in 1.3.0 and newer.  Without
> this patch we get a build error for lto-compress.c when using an old zstd
> version.
>
> OK?

OK

> Jim
>
>         Backported from master:
>         2020-09-29  Jim Wilson  <jimw@sifive.com>
>
>         gcc/
>         PR bootstrap/97183
>         * configure.ac (gcc_cv_header_zstd_h): Check ZSTD_VERISON_NUMBER.
>         * configure: Regenerated.
> ---
>  gcc/configure    | 11 ++++++++---
>  gcc/configure.ac |  7 ++++++-
>  2 files changed, 14 insertions(+), 4 deletions(-)
>
> diff --git a/gcc/configure b/gcc/configure
> index eb6061c1631..b4088d8fd1e 100755
> --- a/gcc/configure
> +++ b/gcc/configure
> @@ -10024,9 +10024,14 @@ $as_echo_n "checking for zstd.h... " >&6; }
>  if ${gcc_cv_header_zstd_h+:} false; then :
>    $as_echo_n "(cached) " >&6
>  else
> -  cat confdefs.h - <<_ACEOF >conftest.$ac_ext
> +  # We require version 1.3.0 or later.  This is the first version that has
> +# ZSTD_getFrameContentSize.
> +cat confdefs.h - <<_ACEOF >conftest.$ac_ext
>  /* end confdefs.h.  */
>  #include <zstd.h>
> +#if ZSTD_VERSION_NUMBER < 10300
> +#error "need zstd 1.3.0 or better"
> +#endif
>  int
>  main ()
>  {
> @@ -19015,7 +19020,7 @@ else
>    lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2
>    lt_status=$lt_dlunknown
>    cat > conftest.$ac_ext <<_LT_EOF
> -#line 19018 "configure"
> +#line 19023 "configure"
>  #include "confdefs.h"
>
>  #if HAVE_DLFCN_H
> @@ -19121,7 +19126,7 @@ else
>    lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2
>    lt_status=$lt_dlunknown
>    cat > conftest.$ac_ext <<_LT_EOF
> -#line 19124 "configure"
> +#line 19129 "configure"
>  #include "confdefs.h"
>
>  #if HAVE_DLFCN_H
> diff --git a/gcc/configure.ac b/gcc/configure.ac
> index 715fcba0482..070b9c6c497 100644
> --- a/gcc/configure.ac
> +++ b/gcc/configure.ac
> @@ -1382,8 +1382,13 @@ LDFLAGS="$LDFLAGS $ZSTD_LDFLAGS"
>
>  AC_MSG_CHECKING(for zstd.h)
>  AC_CACHE_VAL(gcc_cv_header_zstd_h,
> +# We require version 1.3.0 or later.  This is the first version that has
> +# ZSTD_getFrameContentSize.
>  [AC_COMPILE_IFELSE([AC_LANG_PROGRAM(
> -[[#include <zstd.h>]])],
> +[[#include <zstd.h>
> +#if ZSTD_VERSION_NUMBER < 10300
> +#error "need zstd 1.3.0 or better"
> +#endif]])],
>    [gcc_cv_header_zstd_h=yes],
>    [gcc_cv_header_zstd_h=no])])
>  AC_MSG_RESULT($gcc_cv_header_zstd_h)
> --
> 2.17.1
>


More information about the Gcc-patches mailing list