Hi Richard, on 2023/10/20 06:12, Richard Sandiford wrote: > "Kewen.Lin" <li...@linux.ibm.com> writes: >> Hi, >> >> As comments[1][2], this patch is to change the costing way >> on some adjacent vector loads/stores from costing one by >> one to costing them together with the total number once. >> >> It helps to fix the exposed regression PR111784 on aarch64, >> as aarch64 specific costing could make different decisions >> according to the different costing ways (counting with total >> number vs. counting one by one). Based on a reduced test >> case from PR111784, only considering vec_num can fix the >> regression already, but vector loads/stores in regard to >> ncopies are also adjacent accesses, so they are considered >> as well. >> >> btw, this patch leaves the costing on dr_explicit_realign >> and dr_explicit_realign_optimized alone to make it simple. >> The costing way change can cause the differences for them >> since there is one costing depending on targetm.vectorize. >> builtin_mask_for_load and it's costed according to the >> calling times. IIUC, these two dr_alignment_support are >> mainly used for old Power? (only having 16 bytes aligned >> vector load/store but no unaligned vector load/store). >> >> Bootstrapped and regtested on x86_64-redhat-linux, >> aarch64-linux-gnu, powerpc64-linux-gnu P{7,8,9} >> and powerpc64le-linux-gnu P{8,9,10}. >> >> Is it ok for trunk? >> >> [1] https://gcc.gnu.org/pipermail/gcc-patches/2023-September/630742.html >> [2] https://gcc.gnu.org/pipermail/gcc-patches/2023-September/630744.html >> >> BR, >> Kewen >> ----- >> gcc/ChangeLog: >> >> * tree-vect-stmts.cc (vectorizable_store): Adjust costing way for >> adjacent vector stores, by costing them with the total number >> rather than costing them one by one. >> (vectorizable_load): Adjust costing way for adjacent vector >> loads, by costing them with the total number rather than costing >> them one by one. > > OK. Thanks for doing this! Like Richard says, the way that the aarch64 > cost hooks rely on the count isn't super robust, but I think it's the > best we can easily do in the circumstances. Hopefully costing will > become much easier once the non-SLP representation goes away.
Looking forward to that, thanks for the review! Committed in r14-4842. BR, Kewen