Hi Christophe,

> >
> > gcc/testsuite
> > 2017-09-01  Tamar Christina  <tamar.christ...@arm.com>
> >
> >         * lib/target-supports.exp
> >         (check_effective_target_arm_v8_2a_dotprod_neon_ok_nocache):
> New.
> >         (check_effective_target_arm_v8_2a_dotprod_neon_ok): New.
> >         (add_options_for_arm_v8_2a_dotprod_neon): New.
> >         (check_effective_target_arm_v8_2a_dotprod_neon_hw): New.
> >         (check_effective_target_vect_sdot_qi): New.
> >         (check_effective_target_vect_udot_qi): New.
> >         * gcc.target/arm/simd/vdot-exec.c: New.
> 
> Aren't you defining twice P() and ARR() in vdot-exec.c ?
> I'd expect a preprocessor error, did I read too quickly?
>

Yes they are defined twice but they're not redefined, all the definitions
are exactly the same so the pre-processor doesn't care. I can leave only
one if this is confusing.
 
> 
> Thanks,
> 
> Christophe
> 
> >         * gcc.target/aarch64/advsimd-intrinsics/vdot-exec.c: New.
> >         * gcc/doc/sourcebuild.texi: Document arm_v8_2a_dotprod_neon.
> >
> > --

Reply via email to