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.