From: Richard Biener Date: Fri, 15 Jan 2021 13:20:43 +0000 (+0100) Subject: tree-optimization/96376 - do not check alignment for invariant loads X-Git-Tag: upstream/12.2.0~10237 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=446703ccc2ce09e9710964f781a21030bf3c2d54;p=platform%2Fupstream%2Fgcc.git tree-optimization/96376 - do not check alignment for invariant loads The testcases show that we fail to disregard alignment for invariant loads. The patch handles them like we handle gather and scatter. 2021-01-15 Richard Biener PR tree-optimization/96376 * tree-vect-stmts.c (get_load_store_type): Disregard alignment for VMAT_INVARIANT. --- diff --git a/gcc/tree-vect-stmts.c b/gcc/tree-vect-stmts.c index 4d72c4d..f180ced 100644 --- a/gcc/tree-vect-stmts.c +++ b/gcc/tree-vect-stmts.c @@ -2378,19 +2378,26 @@ get_load_store_type (vec_info *vinfo, stmt_vec_info stmt_info, else { int cmp = compare_step_with_zero (vinfo, stmt_info); - if (cmp < 0) - *memory_access_type = get_negative_load_store_type - (vinfo, stmt_info, vectype, vls_type, ncopies); - else if (cmp == 0) + if (cmp == 0) { gcc_assert (vls_type == VLS_LOAD); *memory_access_type = VMAT_INVARIANT; + /* Invariant accesses perform only component accesses, alignment + is irrelevant for them. */ + *alignment_support_scheme = dr_unaligned_supported; } else - *memory_access_type = VMAT_CONTIGUOUS; - *alignment_support_scheme - = vect_supportable_dr_alignment (vinfo, - STMT_VINFO_DR_INFO (stmt_info), false); + { + if (cmp < 0) + *memory_access_type = get_negative_load_store_type + (vinfo, stmt_info, vectype, vls_type, ncopies); + else + *memory_access_type = VMAT_CONTIGUOUS; + *alignment_support_scheme + = vect_supportable_dr_alignment (vinfo, + STMT_VINFO_DR_INFO (stmt_info), + false); + } } if ((*memory_access_type == VMAT_ELEMENTWISE