diff options
| author | Richard Biener <rguenther@suse.de> | 2024-09-12 11:31:59 +0200 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2024-09-12 13:02:23 +0200 |
| commit | 412c156d78c764d4aec3e94469ba5a4c068cee4c (patch) | |
| tree | 1f7376c718017263c6203482c26e61f1f089067d /gcc/tree-vect-loop.cc | |
| parent | 44058b847145166715f15e49fa8854f30e852f24 (diff) | |
| download | gcc-412c156d78c764d4aec3e94469ba5a4c068cee4c.zip gcc-412c156d78c764d4aec3e94469ba5a4c068cee4c.tar.gz gcc-412c156d78c764d4aec3e94469ba5a4c068cee4c.tar.bz2 | |
Abort loop SLP analysis quicker
As we can't cope with removed SLP instances during analysis there's
no point in doing that or even continuing analysis of SLP instances
after a failure. The following makes us abort early.
* tree-vect-slp.cc (vect_slp_analyze_operations): When
doing loop analysis fail after the first failed SLP
instance. Only remove instances when doing BB vectorization.
* tree-vect-loop.cc (vect_analyze_loop_2): Check whether
vect_slp_analyze_operations failed instead of checking
the number of SLP instances remaining.
Diffstat (limited to 'gcc/tree-vect-loop.cc')
| -rw-r--r-- | gcc/tree-vect-loop.cc | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/gcc/tree-vect-loop.cc b/gcc/tree-vect-loop.cc index 65d7ed5..cc15492 100644 --- a/gcc/tree-vect-loop.cc +++ b/gcc/tree-vect-loop.cc @@ -2947,12 +2947,10 @@ start_over: if (slp) { - /* Analyze operations in the SLP instances. Note this may - remove unsupported SLP instances which makes the above - SLP kind detection invalid. */ - unsigned old_size = LOOP_VINFO_SLP_INSTANCES (loop_vinfo).length (); - vect_slp_analyze_operations (loop_vinfo); - if (LOOP_VINFO_SLP_INSTANCES (loop_vinfo).length () != old_size) + /* Analyze operations in the SLP instances. We can't simply + remove unsupported SLP instances as this makes the above + SLP kind detection invalid and might also affect the VF. */ + if (! vect_slp_analyze_operations (loop_vinfo)) { ok = opt_result::failure_at (vect_location, "unsupported SLP instances\n"); |