Uros,
Could you please merge the patch into mainline?
Thank you
Sergey

-----Original Message-----
From: Uros Bizjak [mailto:ubiz...@gmail.com] 
Sent: Tuesday, September 19, 2017 6:17 PM
To: Shalnov, Sergey <sergey.shal...@intel.com>
Cc: gcc-patches@gcc.gnu.org; kirill.yuk...@gmail.com; Koval, Julia 
<julia.ko...@intel.com>; Senkevich, Andrew <andrew.senkev...@intel.com>
Subject: Re: [PATCH, i386] Enable option -mprefer-avx256 added for Intel AVX512 
configuration

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