on 2023/7/21 19:49, Richard Biener wrote: > On Fri, Jul 21, 2023 at 8:08 AM Kewen.Lin <li...@linux.ibm.com> wrote: >> >> Hi, >> >> The function vect_update_epilogue_niters which has been >> removed by r14-2281 has some code taking care of that if >> there is only one scalar iteration left for epilogue then >> we won't try to vectorize it any more. >> >> Although costing should be able to care about it eventually, >> I think we still want this special casing without costing >> enabled, so this patch is to add it back in function >> vect_analyze_loop_costing, and make it more general for >> both main and epilogue loops as Richi suggested, it can fix >> some exposed failures on Power10: >> >> - gcc.target/powerpc/p9-vec-length-epil-{1,8}.c >> - gcc.dg/vect/slp-perm-{1,5,6,7}.c >> >> Bootstrapped and regtested on x86_64-redhat-linux, >> aarch64-linux-gnu, powerpc64-linux-gnu P8/P9 and >> powerpc64le-linux-gnu P9/P10. >> >> Is it ok for trunk? > > OK. >
Thanks Richi, pushed as r14-2736. BR, Kewen