On 11 January 2018 at 17:29, Kyrill Tkachov <kyrylo.tkac...@foss.arm.com> wrote: > Hi Christophe, > > On 11/01/18 16:21, Christophe Lyon wrote: >> >> On 11 January 2018 at 16:33, Kyrill Tkachov >> <kyrylo.tkac...@foss.arm.com> wrote: >>> >>> Hi all, >>> >>> This recently added test fails on arm. We haven't implemented these >>> intrinsics for arm >>> (any volunteers?) so for now let's XFAIL these on that target. >>> Also, the float64 versions of these intrinsics are not supposed to be >>> available on arm >>> so this patch slightly adjusts the test to not include them for aarch32. >>> In any case the entire test is XFAILed on arm, so this doesn't have any >>> noticeable >>> effect. >>> >>> The same number of tests (PASS) still occur on aarch64 but now they >>> appear >>> as XFAIL >>> rather than FAIL on arm. >>> >>> Ok for trunk? (from an aarch64 perspective). >>> >> I think Kugan was planning to skip them on arm instead? > > > These intrinsics are specified to exist on arm as well: > (search on https://developer.arm.com/technologies/neon/intrinsics) > except the float64 ones, so from a purist perspective the tests > should be passing on arm, but we know the reason why they're not > (they are not implemented there), which is the rationale for XFAIL. > Does XFAILing them interact badly with any bookkeeping that your > testing system does? > No problem at all, I mainly cc'ed Kugan to make sure he is aware of your patch.
> Thanks, > Kyrill > > >> >>> Thanks, >>> Kyrill >>> >>> 2018-01-11 Kyrylo Tkachov <kyrylo.tkac...@arm.com> >>> >>> * gcc.target/aarch64/advsimd-intrinsics/vld1x2.c: Make float64 >>> tests specific to aarch64. XFAIL test on arm. > >
