Re: [PATCH] Turn vect_create_addr_base_for_vector_ref offset into a byte offset

[Bernhard Reutner-Fischer via Gcc-patches]

On 26 October 2021 10:58:50 CEST, Richard Biener via Gcc-patches 
<gcc-patches@gcc.gnu.org> wrote:

>@@ -2006,6 +2011,7 @@ get_negative_load_store_type (vec_info *vinfo,
>       dump_printf_loc (MSG_MISSED_OPTIMIZATION, vect_location,
>                        "negative step but alignment required.\n");
>       return VMAT_ELEMENTWISE;
>+      *poffset = 0;

Would you want to move this to before the return?
thanks,
>     }
> 
>   if (vls_type == VLS_STORE_INVARIANT)
>@@ -2014,7 +2020,6 @@ get_negative_load_store_type (vec_info *vinfo,
>       dump_printf_loc (MSG_NOTE, vect_location,
>                        "negative step with invariant source;"
>                        " no permute needed.\n");
>-      *poffset = -TYPE_VECTOR_SUBPARTS (vectype) + 1;
>       return VMAT_CONTIGUOUS_DOWN;
>     }