diff options
author | Tamar Christina <tamar.christina@arm.com> | 2025-08-19 10:17:17 +0100 |
---|---|---|
committer | Tamar Christina <tamar.christina@arm.com> | 2025-08-19 10:17:17 +0100 |
commit | 08cdd61e70c9c33a7a33b9c656cbc2ccb2914bd1 (patch) | |
tree | 748a9ffde80045c3f7b4ca3f8751c07cb4778fa6 /gcc/testsuite/gcc.dg/cpp/gnu11-elifdef-2.c | |
parent | 4982644625dc5a46e959ecf8167641f6ae21f76d (diff) | |
download | gcc-08cdd61e70c9c33a7a33b9c656cbc2ccb2914bd1.zip gcc-08cdd61e70c9c33a7a33b9c656cbc2ccb2914bd1.tar.gz gcc-08cdd61e70c9c33a7a33b9c656cbc2ccb2914bd1.tar.bz2 |
middle-end: Fix costing hooks of various vectorizable_* [PR121536]
commit g:1786be14e94bf1a7806b9dc09186f021737f0227 stops storing in
STMT_VINFO_VECTYPE the vectype of the current stmt being vectorized and instead
requires the use of SLP_TREE_VECTYPE for everything but data-refs.
However contrary to what the commit says not all usages of STMT_VINFO_VECTYPE
have been purged from vectorizable_* as the costing hooks which don't pass the
SLP tree as an agrument will extract vectype using STMT_VINFO_VECTYPE.
This results in no vector type being passed to the backends and results in a few
costing test failures in AArch64.
This commit replaces the last few cases I could find, all except for in
vectorizable_reduction when single_defuse_cycle where the stmt being costed is
not the representative of the PHI in the SLP tree but rather the out of tree
reduction statement. So I've left that alone, but it does mean vectype is NULL.
Most likely this needs to use the overload where we pass an explicit vectype but
I wasn't sure so left it for now.
gcc/ChangeLog:
PR target/121536
* tree-vect-loop.cc (vectorizable_phi, vectorizable_recurr,
vectorizable_nonlinear_induction, vectorizable_induction): Pass slp_node
instead of stmt_info to record_stmt_cost.
Diffstat (limited to 'gcc/testsuite/gcc.dg/cpp/gnu11-elifdef-2.c')
0 files changed, 0 insertions, 0 deletions