This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] xmmintrin.h needs to use __vector __bool everywhere
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, David Edelsohn <dje dot gcc at gmail dot com>
- Date: Tue, 27 Mar 2018 19:14:48 -0500
- Subject: Re: [PATCH, rs6000] xmmintrin.h needs to use __vector __bool everywhere
- References: <1f4fbc22-a8bf-7eb3-b02c-2d4ccbc6c8f4@linux.vnet.ibm.com>
Hi Bill,
On Tue, Mar 27, 2018 at 04:10:00PM -0500, Bill Schmidt wrote:
> The xmmintrin.h compatibility header embeds altivec.h to use the Power
> vector intrinsics. However, it needs to be careful not to use the "bool"
> keyword with vectors so the headers don't cause potential problems in C++
> and C11 code when using strict-ANSI. I noticed a few cases where this was
> happening. They haven't caused trouble yet, but it's just a matter of time.
> This patch cleans those up.
>
> Bootstrapped and tested on powerpc64le-unknown-linux-gnu. Is this okay
> for trunk?
Yes please. Thanks!
Segher
> 2018-03-27 Bill Schmidt <wschmidt@linux.vnet.ibm.com>
>
> * config/rs6000/xmmintrin.h (_mm_max_pi16): Use __vector __bool
> instead of __vector bool.
> (_mm_max_pu8): Likewise.
> (_mm_min_pi16): Likewise.