On Thu, 10 Aug 2023, Robin Dapp wrote: > > Hmm, I think VEC_EXTRACT and VEC_SET should be ECF_CONST. Maybe the > > GIMPLE ISEL > > comments do not match the implementation, but then that should be fixed? > > > > /* Expand all ARRAY_REF(VIEW_CONVERT_EXPR) gimple assignments into calls > > to > > internal function based on vector type of selected expansion. > > > > For vec_set: > > > > VIEW_CONVERT_EXPR<int[4]>(u)[_1] = i_4(D); > > => > > _7 = u; > > _8 = .VEC_SET (_7, i_4(D), _1); > > u = _8; > > > > For vec_extract: > > > > _3 = VIEW_CONVERT_EXPR<intD.1[4]>(vD.2208)[idx_2(D)]; > > => > > _4 = vD.2208; > > _3 = .VEC_EXTRACT (_4, idx_2(D)); */ > > > > I probably just forgot to set ECF_CONST in the recent isel patch > for vec_extract.
I'm testing a patch adjusting a few IFNs where that was missed. Richard.