On Wed, Jan 15, 2014 at 10:56:35AM -0700, Jeff Law wrote:
> On 01/13/14 00:34, Jakub Jelinek wrote:
> >On Sun, Jan 12, 2014 at 11:21:59PM -0700, Jeff Law wrote:
> >>--- a/gcc/ree.c
> >>+++ b/gcc/ree.c
> >>@@ -297,6 +297,13 @@ combine_set_extension (ext_cand *cand, rtx curr_insn,
> >>rtx *orig_set)
> >> else
> >> new_reg = gen_rtx_REG (cand->mode, REGNO (SET_DEST (*orig_set)));
> >>
> >>+ /* We're going to be widening the result of DEF_INSN, ensure that doing
> >>so
> >>+ doesn't change the number of hard registers needed for the result. */
> >>+ if (HARD_REGNO_NREGS (REGNO (new_reg), cand->mode)
> >>+ != HARD_REGNO_NREGS (REGNO (SET_SRC (*orig_set)),
> >
> >Note you can use orig_src instead of SET_SRC (*orig_set) here.
> Yea. A little manual CSE never hurts. Fixed.
The patch is ok with that fix then. Thanks.
Jakub
