https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98113
Kewen Lin <linkw at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |rguenther at suse dot de
Last reconfirmed| |2020-12-03
Status|UNCONFIRMED |ASSIGNED
Ever confirmed|0 |1
--- Comment #1 from Kewen Lin <linkw at gcc dot gnu.org> ---
(In reply to Ilya Leoshkevich from comment #0)
> s390's vxe/popcount-1.c began to fail after PR96789 fix.
Sorry to see this regression.
...
>
> that is, replaced a sequence of stores with a sequence of
> BIT_INSERT_EXPRs.
>
> slp1 now says: "missed: not vectorized: no grouped stores in basic
> block", presumably because it doesn't understand BIT_INSERT_EXPRs.
Yes, currently slp instance is built from group stores (also CTOR), I expect
Richi's ongoing slp rework will extend it to support group loads. For this
BIT_INSERT_EXPR, I guess we can extend vector CTOR handling to cover this if
the BIT_INSERT_EXPR chain can only and fully cover all lanes which is
equivalent to vector CTOR.
CC @Richi. Hi Richi, what do you think of this?
