Committed as passed both bootstrap and regression tests.

Pan

-----Original Message-----
From: Gcc-patches <[email protected]> On Behalf 
Of Bernhard Reutner-Fischer via Gcc-patches
Sent: Friday, June 23, 2023 5:39 PM
To: Richard Sandiford <[email protected]>
Cc: [email protected]; [email protected]; [email protected]
Subject: Re: [PATCH V6] VECT: Apply LEN_MASK_{LOAD,STORE} into vectorizer

On 23 June 2023 10:03:45 CEST, Richard Sandiford <[email protected]> 
wrote:

>> Fuse the block below into the one above as the condition seems to be 
>> identical?
>
>Yeah, true, but I think the idea is that the code above “Arguments are
>ready” is calculating argument values, and the code after it is creating
>code.  These are two separate steps, and the fact that the two final_len
>blocks end up being consecutive is something of a coincidence.
>
>So personally I think we should keep the structure in the patch.

Sure, works for me.
thanks,

Reply via email to