diff options
| author | Richard Biener <rguenther@suse.de> | 2021-07-19 13:29:16 +0200 |
|---|---|---|
| committer | Richard Biener <rguenther@suse.de> | 2021-07-19 14:21:42 +0200 |
| commit | 8df3ee8f7d85d0708f3c3ca96b55c9230c2ae9f0 (patch) | |
| tree | f00c8a35a0c22853e651aacd0b5128dadd0fd31b /gcc | |
| parent | 9f2bc5077debef2b046b6c10d38591ac324ad8b5 (diff) | |
| download | gcc-8df3ee8f7d85d0708f3c3ca96b55c9230c2ae9f0.zip gcc-8df3ee8f7d85d0708f3c3ca96b55c9230c2ae9f0.tar.gz gcc-8df3ee8f7d85d0708f3c3ca96b55c9230c2ae9f0.tar.bz2 | |
tree-optimization/101505 - properly determine stmt precision for PHIs
Loop vectorization pattern recog fails to walk PHIs when determining
stmt precisions. This fails to recognize non-mask uses for bools
in PHIs and outer loop vectorization.
2021-07-19 Richard Biener <rguenther@suse.de>
PR tree-optimization/101505
* tree-vect-patterns.c (vect_determine_precisions): Walk
PHIs also for loop vectorization.
* gcc.dg/vect/pr101505.c: New testcase.
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/testsuite/gcc.dg/vect/pr101505.c | 16 | ||||
| -rw-r--r-- | gcc/tree-vect-patterns.c | 14 |
2 files changed, 30 insertions, 0 deletions
diff --git a/gcc/testsuite/gcc.dg/vect/pr101505.c b/gcc/testsuite/gcc.dg/vect/pr101505.c new file mode 100644 index 0000000..e2b8945 --- /dev/null +++ b/gcc/testsuite/gcc.dg/vect/pr101505.c @@ -0,0 +1,16 @@ +/* { dg-do compile } */ +/* { dg-additional-options "-O1" } */ + +int n2; + +__attribute__ ((simd)) char +w7 (void) +{ + short int xb = n2; + int qp; + + for (qp = 0; qp < 2; ++qp) + xb = xb < 1; + + return xb; +} diff --git a/gcc/tree-vect-patterns.c b/gcc/tree-vect-patterns.c index c249444..44f6c9b 100644 --- a/gcc/tree-vect-patterns.c +++ b/gcc/tree-vect-patterns.c @@ -5355,6 +5355,13 @@ vect_determine_precisions (vec_info *vinfo) for (unsigned int i = 0; i < nbbs; i++) { basic_block bb = bbs[i]; + for (auto gsi = gsi_start_phis (bb); + !gsi_end_p (gsi); gsi_next (&gsi)) + { + stmt_vec_info stmt_info = vinfo->lookup_stmt (gsi.phi ()); + if (stmt_info) + vect_determine_mask_precision (vinfo, stmt_info); + } for (auto si = gsi_start_bb (bb); !gsi_end_p (si); gsi_next (&si)) if (!is_gimple_debug (gsi_stmt (si))) vect_determine_mask_precision @@ -5368,6 +5375,13 @@ vect_determine_precisions (vec_info *vinfo) if (!is_gimple_debug (gsi_stmt (si))) vect_determine_stmt_precisions (vinfo, vinfo->lookup_stmt (gsi_stmt (si))); + for (auto gsi = gsi_start_phis (bb); + !gsi_end_p (gsi); gsi_next (&gsi)) + { + stmt_vec_info stmt_info = vinfo->lookup_stmt (gsi.phi ()); + if (stmt_info) + vect_determine_stmt_precisions (vinfo, stmt_info); + } } } else |