On Tue, Oct 27, 2015 at 5:40 AM, Alan Lawrence <alan.lawre...@arm.com> wrote: > --in-reply-to > <cafiyyc3tepgber2jqc8-x_ij4ghtjjoxfzffcnyzhxhgqbe...@mail.gmail.com> > > On 26/10/15 08:58, Richard Biener wrote: >> >> On Fri, Oct 23, 2015 at 5:15 PM, Alan Lawrence <alan.lawre...@arm.com> wrote: >>> + chrec2 = fold_build2 (LSHIFT_EXPR, TREE_TYPE (rhs1), >>> + build_int_cst (TREE_TYPE (rhs1), 1), >> >> 'type' instead of TREE_TYPE (rhs1) > > I presume you mean the first of the two (allowing removal of the > chrec_convert), > and that I keep the second TREE_TYPE (rhs1), consistent with your previous > observation that I should do the multiply in the type of rhs1. (This appears > correct looking at e.g. gcc.target/i386/avx2-vpsllwi-2.c) > > Hence, I've committed the attached as r229437. >
It caused: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68112 -- H.J.
