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.