diff options
author | Richard Biener <rguenther@suse.de> | 2020-11-18 09:36:57 +0100 |
---|---|---|
committer | Richard Biener <rguenther@suse.de> | 2021-09-27 10:24:12 +0200 |
commit | 6390c5047adb75960f86d56582e6322aaa4d9281 (patch) | |
tree | b3f3817652f28f26ddf2bc354c87f37c68736fb4 /gcc/tree-vectorizer.h | |
parent | e7b8d7020052110e5717230104e647f6235dd2c1 (diff) | |
download | gcc-6390c5047adb75960f86d56582e6322aaa4d9281.zip gcc-6390c5047adb75960f86d56582e6322aaa4d9281.tar.gz gcc-6390c5047adb75960f86d56582e6322aaa4d9281.tar.bz2 |
Allow different vector types for stmt groups
This allows vectorization (in practice non-loop vectorization) to
have a stmt participate in different vector type vectorizations.
It allows us to remove vect_update_shared_vectype and replace it
by pushing/popping STMT_VINFO_VECTYPE from SLP_TREE_VECTYPE around
vect_analyze_stmt and vect_transform_stmt.
For data-ref the situation is a bit more complicated since we
analyze alignment info with a specific vector type in mind which
doesn't play well when that changes.
So the bulk of the change is passing down the actual vector type
used for a vectorized access to the various accessors of alignment
info, first and foremost dr_misalignment but also aligned_access_p,
known_alignment_for_access_p, vect_known_alignment_in_bytes and
vect_supportable_dr_alignment. I took the liberty to replace
ALL_CAPS macro accessors with the lower-case function invocations.
The actual changes to the behavior are in dr_misalignment which now
is the place factoring in the negative step adjustment as well as
handling alignment queries for a vector type with bigger alignment
requirements than what we can (or have) analyze(d).
vect_slp_analyze_node_alignment makes use of this and upon receiving
a vector type with a bigger alingment desire re-analyzes the DR
with respect to it but keeps an older more precise result if possible.
In this context it might be possible to do the analysis just once
but instead of analyzing with respect to a specific desired alignment
look for the biggest alignment we can compute a not unknown alignment.
The ChangeLog includes the functional changes but not the bulk due
to the alignment accessor API changes - I hope that's something good.
2021-09-17 Richard Biener <rguenther@suse.de>
PR tree-optimization/97351
PR tree-optimization/97352
PR tree-optimization/82426
* tree-vectorizer.h (dr_misalignment): Add vector type
argument.
(aligned_access_p): Likewise.
(known_alignment_for_access_p): Likewise.
(vect_supportable_dr_alignment): Likewise.
(vect_known_alignment_in_bytes): Likewise. Refactor.
(DR_MISALIGNMENT): Remove.
(vect_update_shared_vectype): Likewise.
* tree-vect-data-refs.c (dr_misalignment): Refactor, handle
a vector type with larger alignment requirement and apply
the negative step adjustment here.
(vect_calculate_target_alignment): Remove.
(vect_compute_data_ref_alignment): Get explicit vector type
argument, do not apply a negative step alignment adjustment
here.
(vect_slp_analyze_node_alignment): Re-analyze alignment
when we re-visit the DR with a bigger desired alignment but
keep more precise results from smaller alignments.
* tree-vect-slp.c (vect_update_shared_vectype): Remove.
(vect_slp_analyze_node_operations_1): Do not update the
shared vector type on stmts.
* tree-vect-stmts.c (vect_analyze_stmt): Push/pop the
vector type of an SLP node to the representative stmt-info.
(vect_transform_stmt): Likewise.
* gcc.target/i386/vect-pr82426.c: New testcase.
* gcc.target/i386/vect-pr97352.c: Likewise.
Diffstat (limited to 'gcc/tree-vectorizer.h')
-rw-r--r-- | gcc/tree-vectorizer.h | 32 |
1 files changed, 15 insertions, 17 deletions
diff --git a/gcc/tree-vectorizer.h b/gcc/tree-vectorizer.h index c4c5678..ed4a7ff 100644 --- a/gcc/tree-vectorizer.h +++ b/gcc/tree-vectorizer.h @@ -1606,11 +1606,8 @@ set_dr_misalignment (dr_vec_info *dr_info, int val) dr_info->misalignment = val; } -extern int dr_misalignment (dr_vec_info *dr_info); +extern int dr_misalignment (dr_vec_info *dr_info, tree vectype); -/* Reflects actual alignment of first access in the vectorized loop, - taking into account peeling/versioning if applied. */ -#define DR_MISALIGNMENT(DR) dr_misalignment (DR) #define SET_DR_MISALIGNMENT(DR, VAL) set_dr_misalignment (DR, VAL) /* Only defined once DR_MISALIGNMENT is defined. */ @@ -1630,35 +1627,37 @@ set_dr_target_alignment (dr_vec_info *dr_info, poly_uint64 val) } #define SET_DR_TARGET_ALIGNMENT(DR, VAL) set_dr_target_alignment (DR, VAL) -/* Return true if data access DR_INFO is aligned to its target alignment - (which may be less than a full vector). */ +/* Return true if data access DR_INFO is aligned to the targets + preferred alignment for VECTYPE (which may be less than a full vector). */ static inline bool -aligned_access_p (dr_vec_info *dr_info) +aligned_access_p (dr_vec_info *dr_info, tree vectype) { - return (DR_MISALIGNMENT (dr_info) == 0); + return (dr_misalignment (dr_info, vectype) == 0); } -/* Return TRUE if the alignment of the data access is known, and FALSE +/* Return TRUE if the (mis-)alignment of the data access is known with + respect to the targets preferred alignment for VECTYPE, and FALSE otherwise. */ static inline bool -known_alignment_for_access_p (dr_vec_info *dr_info) +known_alignment_for_access_p (dr_vec_info *dr_info, tree vectype) { - return (DR_MISALIGNMENT (dr_info) != DR_MISALIGNMENT_UNKNOWN); + return (dr_misalignment (dr_info, vectype) != DR_MISALIGNMENT_UNKNOWN); } /* Return the minimum alignment in bytes that the vectorized version of DR_INFO is guaranteed to have. */ static inline unsigned int -vect_known_alignment_in_bytes (dr_vec_info *dr_info) +vect_known_alignment_in_bytes (dr_vec_info *dr_info, tree vectype) { - if (DR_MISALIGNMENT (dr_info) == DR_MISALIGNMENT_UNKNOWN) + int misalignment = dr_misalignment (dr_info, vectype); + if (misalignment == DR_MISALIGNMENT_UNKNOWN) return TYPE_ALIGN_UNIT (TREE_TYPE (DR_REF (dr_info->dr))); - if (DR_MISALIGNMENT (dr_info) == 0) + else if (misalignment == 0) return known_alignment (DR_TARGET_ALIGNMENT (dr_info)); - return DR_MISALIGNMENT (dr_info) & -DR_MISALIGNMENT (dr_info); + return misalignment & -misalignment; } /* Return the behavior of DR_INFO with respect to the vectorization context @@ -1971,7 +1970,7 @@ extern opt_tree vect_get_mask_type_for_stmt (stmt_vec_info, unsigned int = 0); /* In tree-vect-data-refs.c. */ extern bool vect_can_force_dr_alignment_p (const_tree, poly_uint64); extern enum dr_alignment_support vect_supportable_dr_alignment - (vec_info *, dr_vec_info *, bool); + (vec_info *, dr_vec_info *, tree, bool); extern tree vect_get_smallest_scalar_type (stmt_vec_info, tree); extern opt_result vect_analyze_data_ref_dependences (loop_vec_info, unsigned int *); extern bool vect_slp_analyze_instance_dependence (vec_info *, slp_instance); @@ -2110,7 +2109,6 @@ extern bool can_duplicate_and_interleave_p (vec_info *, unsigned int, tree, extern void duplicate_and_interleave (vec_info *, gimple_seq *, tree, const vec<tree> &, unsigned int, vec<tree> &); extern int vect_get_place_in_interleaving_chain (stmt_vec_info, stmt_vec_info); -extern bool vect_update_shared_vectype (stmt_vec_info, tree); extern slp_tree vect_create_new_slp_node (unsigned, tree_code); extern void vect_free_slp_tree (slp_tree); |