On Tue, Sep 19, 2017 at 4:00 PM, Shalnov, Sergey
<sergey.shal...@intel.com> wrote:
> Uros
> Please, find the new patch attached.
> Previously, I thought to change integer part of the " 
> ix86_preferred_simd_mode" function in the same way
> as it done with floating point part. I haven't done it to disturb less code 
> by the patch.
> I completely agree with your proposal, currently the code looks better.
>
> In case of  " ix86_autovectorize_vector_sizes ", I've done changes in the 
> proposed way, but I keep
> algorithm the same as it was in mainline. Please note, the function can 
> return "0" in original version.
> I assume you just propose the "style of changes" in previous message.
>
> If you like to change "ix86_autovectorize_vector_sizes" function 
> algorithmically, I would propose to do
> this in separate patch.

In fact, I was just not careful enough when proposing new style.

The updated patch is OK for mainline.

Thanks,
Uros.

Reply via email to