This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] tree-scalar-evolution.c: Handle LSHIFT by constant
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Alan Lawrence <alan dot lawrence at arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Biener <richard dot guenther at gmail dot com>
- Date: Tue, 27 Oct 2015 15:27:11 -0700
- Subject: Re: [PATCH] tree-scalar-evolution.c: Handle LSHIFT by constant
- Authentication-results: sourceware.org; auth=none
- References: <CAFiYyc3TePGbER2Jqc8-X_ij4GHtjjoxfzFFcNYzHxhGQbE0iQ at mail dot gmail dot com> <1445949611-16406-1-git-send-email-alan dot lawrence at arm dot com>
On Tue, Oct 27, 2015 at 5:40 AM, Alan Lawrence <alan.lawrence@arm.com> wrote:
> --in-reply-to <CAFiYyc3TePGbER2Jqc8-X_ij4GHtjjoxfzFFcNYzHxhGQbE0iQ@mail.gmail.com>
>
> On 26/10/15 08:58, Richard Biener wrote:
>>
>> On Fri, Oct 23, 2015 at 5:15 PM, Alan Lawrence <alan.lawrence@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.