On Fri, Apr 10, 2020 at 11:28 AM Kewen.Lin <li...@linux.ibm.com> wrote:
>
> Hi,
>
> This is one fix following Richi's comments here:
> https://gcc.gnu.org/pipermail/gcc-patches/2020-March/542232.html
>
> I noticed the current half vector support for no peeling gaps
> handled some cases which never check the half size vector
> support.  By further investigation, those cases are safe
> to play without peeling gaps due to ideal alignment.  It
> means they don't require half vector handlings, we should
> avoid to use half vector for them.
>
> The fix is to add alignment check as a part of conditions for
> half vector support avoiding redundant half vector codes.
>
> Bootstrapped/regtested on powerpc64le-linux-gnu P8, while
> aarch64-linux-gnu testing is ongoing.
>
> Is it ok for trunk if all testings are fine?

OK for stage1 (it's just a missed optimization).

Thanks,
Richard.

> BR,
> Kewen
> ----------------
>
> gcc/ChangeLog
>
> 2020-MM-DD  Kewen Lin  <li...@gcc.gnu.org>
>
>         * gcc/tree-vect-stmts.c (vectorizable_load): Check alignment to avoid
>         redundant half vector handlings for no peeling gaps.

Reply via email to