Hi!

On Mon, May 08, 2017 at 09:49:57AM -0500, Steven Munroe wrote:
> Thus I would like to restrict this support to PowerPC
> targets that support VMX/VSX and PowerISA-2.07 (power8) and later.

What happens if you run it on an older machine, or as BE or 32-bit,
or with vectors disabled?

> So I propose to submit a series of patches to implement the PowerPC64LE
> equivalent of a useful subset of the x86 intrinsics. The final size and
> usefulness of this effort is to be determined. The proposal is to
> incrementally port intrinsic header files from the ./config/i386 tree to
> the ./config/rs6000 tree. This naturally provides the same header
> structure and intrinsic names which will simplify code porting.

Yeah.

I'd still like to see these headers moved into some subdir (both in
the source tree and in the installed headers tree), to reduce clutter,
but I understand it's not trivial to do.

> To get the ball rolling I include the BMI intrinsics ported to PowerPC
> for review as they are reasonable size (31 intrinsic implementations).

This is okay for trunk.  Thanks!

> --- gcc/config.gcc    (revision 247616)
> +++ gcc/config.gcc    (working copy)
> @@ -444,7 +444,7 @@ nvptx-*-*)
>       ;;
>  powerpc*-*-*)
>       cpu_type=rs6000
> -     extra_headers="ppc-asm.h altivec.h spe.h ppu_intrinsics.h paired.h
> spu2vmx.h vec_types.h si2vmx.h htmintrin.h htmxlintrin.h"
> +     extra_headers="ppc-asm.h altivec.h spe.h ppu_intrinsics.h paired.h
> spu2vmx.h vec_types.h si2vmx.h htmintrin.h htmxlintrin.h bmi2intrin.h
> bmiintrin.h x86intrin.h"

(Your mail client wrapped this).

Write this on a separate line?  Like
  extra_headers="${extra_headers} htmintrin.h htmxlintrin.h bmi2intrin.h"
(You cannot use += here, pity).


Segher

Reply via email to