This is the mail archive of the
gcc-cvs@gcc.gnu.org
mailing list for the GCC project.
r242085 - /branches/ARM/sve-branch/gcc/combine.c
- From: rsandifo at gcc dot gnu dot org
- To: gcc-cvs at gcc dot gnu dot org
- Date: Fri, 11 Nov 2016 16:57:07 -0000
- Subject: r242085 - /branches/ARM/sve-branch/gcc/combine.c
Author: rsandifo
Date: Fri Nov 11 16:57:07 2016
New Revision: 242085
URL: https://gcc.gnu.org/viewcvs?rev=242085&root=gcc&view=rev
Log:
An alternative fix for PR70944
The transformations made by make_compound_operation apply
only to scalar integer modes. The fix for PR70944 had enforced
that by returning early for vector modes at the top of the
function. However, the function is supposed to be recursive,
so we should continue to look at integer suboperands even if
the outer operation is a vector one.
This patch instead splits out the non-recursive parts
of make_compound_operation into a subroutine and checks
that the mode is a scalar integer before calling it.
The patch was originally written to help with the later
conversion to static type checking of mode classes, but it
also happened to reenable optimisation of things like
vec_duplicate operands.
Note that the gen_lowparts in the PLUS and MINUS cases
were redundant, since new_rtx already had mode "mode"
at those points.
Modified:
branches/ARM/sve-branch/gcc/combine.c