Re: [PATCH] vect: Remove redundant LOOP_VINFO_FULLY_MASKED_P

[Richard Biener via Gcc-patches]

On Wed, Oct 21, 2020 at 10:58 AM Kewen.Lin via Gcc-patches
 wrote:
>
> Hi,
>
> This is a very trivial patch, it's to remove a redundant
> LOOP_VINFO_FULLY_MASKED_P condition check which will be
> checked in vect_use_loop_mask_for_alignment_p.
>
> Is it OK for trunk?

OK.

>
> BR,
> Kewen
> -
> gcc/ChangeLog:
>
> * tree-vect-loop.c (vect_transform_loop): Remove the redundant
> LOOP_VINFO_FULLY_MASKED_P check.
>
> diff --git a/gcc/tree-vect-loop.c b/gcc/tree-vect-loop.c
> index dba230f6320..5e9e25add73 100644
> --- a/gcc/tree-vect-loop.c
> +++ b/gcc/tree-vect-loop.c
> @@ -8913,8 +8913,7 @@ vect_transform_loop (loop_vec_info loop_vinfo, gimple 
> *loop_vectorized_call)
>
>split_edge (loop_preheader_edge (loop));
>
> -  if (LOOP_VINFO_FULLY_MASKED_P (loop_vinfo)
> -  && vect_use_loop_mask_for_alignment_p (loop_vinfo))
> +  if (vect_use_loop_mask_for_alignment_p (loop_vinfo))
>  /* This will deal with any possible peeling.  */
>  vect_prepare_for_masked_peels (loop_vinfo);

[PATCH] vect: Remove redundant LOOP_VINFO_FULLY_MASKED_P

[Kewen.Lin via Gcc-patches]

Hi,

This is a very trivial patch, it's to remove a redundant
LOOP_VINFO_FULLY_MASKED_P condition check which will be
checked in vect_use_loop_mask_for_alignment_p.

Is it OK for trunk?

BR,
Kewen
-
gcc/ChangeLog:

* tree-vect-loop.c (vect_transform_loop): Remove the redundant
LOOP_VINFO_FULLY_MASKED_P check.

diff --git a/gcc/tree-vect-loop.c b/gcc/tree-vect-loop.c
index dba230f6320..5e9e25add73 100644
--- a/gcc/tree-vect-loop.c
+++ b/gcc/tree-vect-loop.c
@@ -8913,8 +8913,7 @@ vect_transform_loop (loop_vec_info loop_vinfo, gimple 
*loop_vectorized_call)

   split_edge (loop_preheader_edge (loop));

-  if (LOOP_VINFO_FULLY_MASKED_P (loop_vinfo)
-  && vect_use_loop_mask_for_alignment_p (loop_vinfo))
+  if (vect_use_loop_mask_for_alignment_p (loop_vinfo))
 /* This will deal with any possible peeling.  */
 vect_prepare_for_masked_peels (loop_vinfo);