Re: [Mesa-dev] [PATCH 4/7 v2] configure.ac: Introduce HAVE_ARM_ASM/HAVE_AARCH64_ASM and the -D flags.

2017-08-22 Thread Eric Anholt
Emil Velikov writes: > Hi Eric, > > On 10 August 2017 at 23:43, Eric Anholt wrote: > >> +aarch64) >> +DEFINES="$DEFINES -DUSE_AARCH64_ASM" > I cannot see any places where the define is used. > > Am I missing something or there isn't any? Do

Re: [Mesa-dev] [PATCH 4/7 v2] configure.ac: Introduce HAVE_ARM_ASM/HAVE_AARCH64_ASM and the -D flags.

2017-08-22 Thread Emil Velikov
Hi Eric, On 10 August 2017 at 23:43, Eric Anholt wrote: > +aarch64) > +DEFINES="$DEFINES -DUSE_AARCH64_ASM" I cannot see any places where the define is used. Am I missing something or there isn't any? Do you have some WIP patches that make use of it? Thanks Emil

Re: [Mesa-dev] [PATCH 4/7 v2] configure.ac: Introduce HAVE_ARM_ASM/HAVE_AARCH64_ASM and the -D flags.

2017-08-10 Thread Rob Herring
On Thu, Aug 10, 2017 at 5:43 PM, Eric Anholt wrote: > I've been trying to get away without these conditionals in vc4's NEON > code, but it meant compiling extra unused code on x86, and build failing > on ARMv6. > > v2: Use the _arm/_arm64 flags to simplify detection (suggested by

[Mesa-dev] [PATCH 4/7 v2] configure.ac: Introduce HAVE_ARM_ASM/HAVE_AARCH64_ASM and the -D flags.

2017-08-10 Thread Eric Anholt
I've been trying to get away without these conditionals in vc4's NEON code, but it meant compiling extra unused code on x86, and build failing on ARMv6. v2: Use the _arm/_arm64 flags to simplify detection (suggested by Rob), but hide the _arm version under ARCH_ARM_HAVE_NEON to keep from