On Thu, Dec 01, 2016 at 01:34:29PM +0100, Markus Trippelsdorf wrote:
> Hopefully one last patch for UB in combine.c:
> combine.c:12561:14: runtime error: left shift of negative value -9
> Fixed by casting to unsigned, as usual.
> Tested on ppc64le.
> OK for trunk?
Sure, but please fix the indentation of that last new line (and of the
changelog, too, while you're at it ;-) )
> - const_op <<= INTVAL (XEXP (op0, 1));
> + const_op = (unsigned HOST_WIDE_INT) const_op
> + << INTVAL (XEXP (op0, 1));