[PATCH] [6 and trunk] Fix bootstrap with older host compilers

[PATCH] [6 and trunk] Fix bootstrap with older host compilers

[Segher Boessenkool]

We didn't have __builtin_swap16 on all targets before GCC 4.8; hsa-brig
tries to use it if the host GCC is 4.6 or up though, breaking bootstrap.
This trivial patch fixes it.

Noticed on gcc22.

Is this okay for trunk and gcc-6?


Segher


2016-04-15  Segher Boessenkool  <segher@kernel.crashing.org>

	* has-brig.c (lendian16): Don't try to use __builtin_bswap16
	unless compiling with at least GCC-4.8.

---
 gcc/hsa-brig.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/hsa-brig.c b/gcc/hsa-brig.c
index a943e37..9c74b9a 100644
--- a/gcc/hsa-brig.c
+++ b/gcc/hsa-brig.c
@@ -51,7 +51,7 @@ along with GCC; see the file COPYING3.  If not see
 static uint16_t
 lendian16 (uint16_t val)
 {
-#if GCC_VERSION >= 4006
+#if GCC_VERSION >= 4008
 #if __BYTE_ORDER__ == __ORDER_LITTLE_ENDIAN__
   return val;
 #elif __BYTE_ORDER__ == __ORDER_BIG_ENDIAN__
-- 
1.9.3

Re: [PATCH] [6 and trunk] Fix bootstrap with older host compilers

[Richard Biener]

On Fri, Apr 15, 2016 at 8:31 PM, Segher Boessenkool
<segher@kernel.crashing.org> wrote:
> We didn't have __builtin_swap16 on all targets before GCC 4.8; hsa-brig
> tries to use it if the host GCC is 4.6 or up though, breaking bootstrap.
> This trivial patch fixes it.
>
> Noticed on gcc22.
>
> Is this okay for trunk and gcc-6?

Ok.

Thanks,
Richard.

>
> Segher
>
>
> 2016-04-15  Segher Boessenkool  <segher@kernel.crashing.org>
>
>         * has-brig.c (lendian16): Don't try to use __builtin_bswap16
>         unless compiling with at least GCC-4.8.
>
> ---
>  gcc/hsa-brig.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/hsa-brig.c b/gcc/hsa-brig.c
> index a943e37..9c74b9a 100644
> --- a/gcc/hsa-brig.c
> +++ b/gcc/hsa-brig.c
> @@ -51,7 +51,7 @@ along with GCC; see the file COPYING3.  If not see
>  static uint16_t
>  lendian16 (uint16_t val)
>  {
> -#if GCC_VERSION >= 4006
> +#if GCC_VERSION >= 4008
>  #if __BYTE_ORDER__ == __ORDER_LITTLE_ENDIAN__
>    return val;
>  #elif __BYTE_ORDER__ == __ORDER_BIG_ENDIAN__
> --
> 1.9.3
>