[PATCH] hsakmt: allow building with gcc 4.x v2

Oded Gabbay oded.gabbay at gmail.com
Sun Apr 17 12:43:11 UTC 2016


On Wed, Mar 30, 2016 at 2:12 AM, Bridgman, John <John.Bridgman at amd.com> wrote:
> The hsakmt code requires C99 compiler support, however gcc 4.x
> defaults to C89 (gcc 5 defaults to C11). v2 patch copies code
> from libdrm, using AC_PROG_CC_C99 and checking success.
>
> v1 used AC_PROG_CC_STDC and did not check C99 was enabled.
>
> Signed-off-by: John Bridgman <john.bridgman at amd.com>
> ---
>  configure.ac | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index b8e9bea..0111067 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -66,6 +66,12 @@ AC_CANONICAL_HOST
>  AC_PROG_AWK
>  test_CFLAGS=${CFLAGS+set} # We may override autoconf default CFLAGS.
>  AC_PROG_CC
> +AC_PROG_CC_C99
> +
> +if test "x$ac_cv_prog_cc_c99" = xno; then
> +       AC_MSG_ERROR([Building hsakmt requires C99 enabled compiler])
> +fi
> +
>  AC_PROG_INSTALL
>  AC_PROG_LIBTOOL
>  AC_PROG_MAKE_SET
> --
> 1.9.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

Thanks John,

Reviewed-by: Oded Gabbay <oded.gabbay at gmail.com>

Can you push it to the repo ?
If you get permission error, we need to set-up permissions.

Oded


More information about the dri-devel mailing list