https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107096
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target| |x86_64-*-*
CC| |rsandifo at gcc dot gnu.org
--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
Take the simplified
void foo (int * __restrict dst, short *src, int n)
{
for (int i = 0; i < n; ++i)
dst[i] = src[2*i] + src[2*i+1];
}
here we get vect_record_loop_mask twice with nvectors == 2 and V16HImode
for the load which populates masks[2]. Then we once get V8SImode but
also with nvectors == 2 which leaves the data unadjusted since it
looks at masks[2] as well but if it were to come first we'd have recorded
a different mask vector type.
The masks seem to be constructed in a way to produce two bits per lane
(but we still apply it naively?!) and the V_C_E does actually look wrong
to me. Huh.
With SVE I seem to get (besides a .LOAD_LANE version) permutes of the
mask vector:
loop_mask_116 = VEC_PERM_EXPR <loop_mask_131, loop_mask_131, { 0,
POLY_INT_CST [4, 4], 1, POLY_INT_CST [5, 4], 2, POLY_INT_CST [6, 4], ... }>;
loop_mask_117 = VEC_PERM_EXPR <loop_mask_131, loop_mask_131, { POLY_INT_CST
[2, 2], POLY_INT_CST [6, 6], POLY_INT_CST [3, 2], POLY_INT_CST [7, 6],
POLY_INT_CST [4, 2], POLY_INT_CST [8, 6], ... }>;
...
vect__69.20_118 = .MASK_LOAD (vectp_src.18_114, 16B, loop_mask_116);
vectp_src.18_119 = vectp_src.18_114 + POLY_INT_CST [8, 8];
vect__69.21_120 = .MASK_LOAD (vectp_src.18_119, 16B, loop_mask_117);
...
.MASK_STORE (vectp_dst.25_129, 32B, loop_mask_131, vect__79.24_125);
so the original mask provider here is the larger element vector type
and the smaller element masks are produced from that. That's something
I'd expect for AVX512 as well, not sure where it goes "wrong".
See PR107093 for a patch implementing WHILE_ULT for AVX512.
