"Kewen.Lin via Gcc-patches" <gcc-patches@gcc.gnu.org> writes:
> @@ -1627,9 +1627,9 @@ vect_analyze_loop_costing (loop_vec_info loop_vinfo)
> class loop *loop = LOOP_VINFO_LOOP (loop_vinfo);
> unsigned int assumed_vf = vect_vf_for_cost (loop_vinfo);
>
> - /* Only fully-masked loops can have iteration counts less than the
> + /* Only partial vectorization loops can have iteration counts less than the
> vectorization factor. */
Maybe:
/* Only loops that can handle partially-populated vectors can have iteration
counts less than the vectorization factor. */
> @@ -2168,10 +2168,10 @@ start_over:
> " support peeling for gaps.\n");
> }
>
> - /* If we're vectorizing an epilogue loop, we either need a fully-masked
> - loop or a loop that has a lower VF than the main loop. */
> + /* If we're vectorizing an epilogue loop, we either need a partial
> + vectorization loop or a loop that has a lower VF than the main loop. */
Maybe:
/* If we're vectorizing an epilogue loop, the vectorized loop either needs
to be able to handle fewer than VF scalars, or needs to have a lower VF
than the main loop. */
> diff --git a/gcc/tree-vectorizer.h b/gcc/tree-vectorizer.h
> index 13e6bf461ab..efcf8e8ae60 100644
> --- a/gcc/tree-vectorizer.h
> +++ b/gcc/tree-vectorizer.h
> @@ -624,8 +624,8 @@ public:
> approach for this loop. */
> bool can_use_partial_vectors_p;
>
> - /* True if have decided to use a fully-masked loop. */
> - bool fully_masked_p;
> + /* True if have decided to use partial vectorization for this loop. */
> + bool using_partial_vectors_p;
Maybe:
/* True if we've decided to use partially-populated vectors, so that the
vector loop can handle fewer than VF scalars. */
OK with those changes, thanks.
Richard
