This is the mail archive of the
mailing list for the GCC project.
r242492 - in /trunk/gcc: ChangeLog combine.c
- From: rsandifo at gcc dot gnu dot org
- To: gcc-cvs at gcc dot gnu dot org
- Date: Wed, 16 Nov 2016 14:20:41 -0000
- Subject: r242492 - in /trunk/gcc: ChangeLog combine.c
Date: Wed Nov 16 14:20:40 2016
New Revision: 242492
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
Note that the gen_lowparts in the PLUS and MINUS cases
were redundant, since new_rtx already had mode "mode"
at those points.
2016-11-15 Richard Sandiford <firstname.lastname@example.org>
Alan Hayward <email@example.com>
David Sherwood <firstname.lastname@example.org>
* combine.c (maybe_swap_commutative_operands): New function.
(combine_simplify_rtx): Use it.
(make_compound_operation_int): New function, split out of...
(make_compound_operation): ...here. Use
maybe_swap_commutative_operands for both.