>> The attached patch v2 use the structure by considering the above >> advice and the (code == CONSTRUCTOR || code == VECTOR_CST) part >> can be shared with VIEW_CONVERT_EXPR handlings as below: >> >> op0 gathering (leave V_C_E in code if it's met) >> >> else if (code == CONSTRUCTOR || code == VECTOR_CST || VIEW_CONVERT_EXPR) >> { >> op1 gathering (leave V_C_E in code2) >> >> if (code == VIEW_CONVERT_EXPR || code2 == VIEW_CONVERT_EXPR) >> do the tricks on arg0/arg1/op2 >> >> the previous handlings on CONSTRUCTOR/VECTOR_CST >> } >> >> Also updated "shrinked" to "shrunk" as Segher pointed out. :-) >> >> Does it look better now? > > Yes. The forwprop changes are OK - I'd still like Richard to > review the vec-perm-indices change. >
Thanks Richi! Hi Richard, Gentle ping for the vec-perm-indices change in case this thread escaped from your radar. https://gcc.gnu.org/pipermail/gcc-patches/2021-May/570240.html BR, Kewen