On 01/24/2018 04:31 AM, Kyrill Tkachov wrote: > Hi all, > > This test fails to optimise away the PLUS reduction in the loop on arm > targets when vectorisation > is not enabled due to absence of SIMD instructions. > From reading the logs and the PR I gather that the presence or absence > of SIMD affects the passing of this test > on other targets as well, as evidenced by the long list of xfail targets. > This list looks quite unwieldy to me, but here is a patch adding > non-NEON arm to that list. > Is this ok? > > Or should we always force -mfpu=neon for arm targets instead? That's > what we do for System Z, > but from a purist perspective the loop has nothing vector-specific in it > so a compiler should be > able to reduce it regardless... > > Thanks, > Kyrill > > 2018-01-24 Kyrylo Tkachov <kyrylo.tkac...@arm.com> > > * gcc.dg/tree-ssa/ssa-dom-cse-2.c: XFAIL on !arm_neon arm targets. It does look rather unwieldy. I wonder if we would be better off manually unrolling the loop and conditionalizing it on some kind of vector support.
But in the immediate term, adding another xfailed target is fine. jeff