On Thu, Jan 21, 2021 at 6:51 PM Segher Boessenkool <seg...@kernel.crashing.org> wrote: > > Hi! > > You never committed 2/4? That makes it harder to review this one :-) > > On Sat, Oct 10, 2020 at 03:08:24AM -0500, Xionghu Luo wrote: > > gcc/ChangeLog: > > > > 2020-10-10 Xionghu Luo <luo...@linux.ibm.com> > > > > * config/rs6000/rs6000-c.c (altivec_resolve_overloaded_builtin): > > Generate ARRAY_REF(VIEW_CONVERT_EXPR) for P8 and later > > platforms. > > * config/rs6000/rs6000.c (rs6000_expand_vector_set_var): Update > > to call different path for P8 and P9. > > (rs6000_expand_vector_set_var_p9): New function. > > (rs6000_expand_vector_set_var_p8): New function. > > > > gcc/testsuite/ChangeLog: > > > > 2020-10-10 Xionghu Luo <luo...@linux.ibm.com> > > > > * gcc.target/powerpc/pr79251.p8.c: New test. > > If testing on P9 LE and P7 BE (32-bit and 64-bit) worked, this is okay > for trunk. Thanks!
This testcase ICEs on AIX. Please fix. This was not tested properly. The new pattern does not have matching target conditions for patterns on which it relies. Thanks, David
