This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][vectorizer][2/2] Hook up mult synthesis logic into vectorisation of mult-by-constant
- From: Rainer Orth <ro at CeBiTec dot Uni-Bielefeld dot DE>
- To: Marc Glisse <marc dot glisse at inria dot fr>
- Cc: Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>, gcc-patches at gcc dot gnu dot org, Richard Biener <rguenther at suse dot de>
- Date: Tue, 05 Jul 2016 13:24:10 +0200
- Subject: Re: [PATCH][vectorizer][2/2] Hook up mult synthesis logic into vectorisation of mult-by-constant
- Authentication-results: sourceware.org; auth=none
- References: <5761570E.2020906@foss.arm.com> <alpine.DEB.2.20.1606152318260.6282@laptop-mg.saclay.inria.fr> <57626C14.3050903@foss.arm.com> <alpine.LSU.2.11.1606280952420.29772@t29.fhfr.qr> <5774D79F.3090101@foss.arm.com> <alpine.LSU.2.11.1607011348480.29772@t29.fhfr.qr> <577B7212.6080809@foss.arm.com> <alpine.DEB.2.20.1607051211190.2466@laptop-mg.saclay.inria.fr>
Marc Glisse <marc.glisse@inria.fr> writes:
> On Tue, 5 Jul 2016, Kyrill Tkachov wrote:
>
>> As for testing I've bootstrapped and tested the patch on aarch64 and
>> x86_64 with synth_shift_p in vect_synth_mult_by_constant hacked to be
>> always true to exercise the paths that synthesize the shift by
>> additions. Marc, could you test this on the sparc targets you care about?
>
> I don't have access to Sparc, you want Rainer here (added in Cc:).
As is, the patch doesn't even build on current mainline:
/vol/gcc/src/hg/trunk/local/gcc/tree-vect-patterns.c:2151:56: error: 'mult_variant' has not been declared
target_supports_mult_synth_alg (struct algorithm *alg, mult_variant var,
^
enum mult_variant is only declared in expmed.c.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University