Re: [PATCH] Changes in mode switching

2012-10-02 Thread Vladimir Yakovlev
2012/9/30 Uros Bizjak : > On Thu, Sep 20, 2012 at 8:35 AM, Uros Bizjak wrote: >> On Thu, Sep 20, 2012 at 8:06 AM, Vladimir Yakovlev >> wrote: >>> The compiler with the patch and without post_reload.patch is built and works >>> successfully. It has the only f

Re: [PATCH] Changes in mode switching

2012-10-02 Thread Vladimir Yakovlev
Will we wait for LRA commit or is it possiple to commit to trank vzeroupper patch now? 2012/10/2 Uros Bizjak : > On Tue, Oct 2, 2012 at 11:35 AM, Vladimir Yakovlev > wrote: >>>>> The compiler with the patch and without post_reload.patch is built and >>>>>

Re: [PATCH, middle-end]: Fix mode-switching MODE_EXIT check with __builtin_apply/__builtin_return

2012-11-05 Thread Vladimir Yakovlev
2010, 2011, 2012 Free Software Foundation, Inc. This file is part of GCC. 2012/11/5 Kaz Kojima : > Uros Bizjak wrote: >> 2012-11-04 Vladimir Yakovlev >> Uros Bizjak >> >> * mode-switching.c (create_pre_exit): Added code for >> may

Re: [PATCH] Vzeroupper placement/47440

2012-11-07 Thread Vladimir Yakovlev
Hello, Thanyou for investigation and fixing the problem. I'll answer on remarks later. Regards, Vladimir 2012/11/7 Jakub Jelinek : > On Tue, Nov 06, 2012 at 02:11:50PM -0800, H.J. Lu wrote: >> On Tue, Nov 6, 2012 at 2:30 AM, Kirill Yukhin >> wrote: >> > Hello, >> >> OK for mainline SVN, pleas

Re: [PATCH] Vzeroupper placement/47440

2012-11-07 Thread Vladimir Yakovlev
you, Vladimir 2012/11/7 Vladimir Yakovlev : > Hello, > > Thanyou for investigation and fixing the problem. I'll answer on remarks > later. > > Regards, > Vladimir > > 2012/11/7 Jakub Jelinek : >> On Tue, Nov 06, 2012 at 02:11:50PM -0800, H.J. Lu wrote: >

Fwd: [off-list] Re: [PATCH] Vzeroupper placement/47440

2012-11-09 Thread Vladimir Yakovlev
-- Forwarded message -- From: Vladimir Yakovlev Date: 2012/11/9 Subject: Re: [off-list] Re: [PATCH] Vzeroupper placement/47440 To: Uros Bizjak Копия: "H.J. Lu" , Igor Zamyatin I did changes that moves vzeroupper insertion after reload 2012-11-09 Vladimi

Re: [off-list] Re: [PATCH] Vzeroupper placement/47440

2012-11-09 Thread Vladimir Yakovlev
i, Nov 9, 2012 at 11:21 AM, Vladimir Yakovlev >> wrote: >>> I did changes that moves vzeroupper insertion after reload >>> >>> 2012-11-09 Vladimir Yakovlev >>> >>> * i386/i386-protos.h (ix86_avx256_optimize_mode_switching): New. >>&

Re: [RFC, x86] Changes for AVX and AVX2 processors

2012-12-28 Thread Vladimir Yakovlev
Hello, processor_alias_table contains the same processor type for all "corei7", "corei7-avx", "core-avx-i" and "core-avx2". At least, it has consequence on checking x86_avx256_split_unaligned_load & ix86_tune_mask: for all these processors it results the same. Moreover we cannot turn new features

[RFC, x86] Changes for AVX and AVX2 processors

2012-12-29 Thread Vladimir Yakovlev
I did changes. Please take a look. 2012/12/29, Uros Bizjak : > On Sat, Dec 29, 2012 at 6:26 AM, Vladimir Yakovlev > wrote: > >> processor_alias_table contains the same processor type for all >> "corei7", "corei7-avx", "core-avx-i" and "

Re: [RFC, x86] Changes for AVX and AVX2 processors

2012-12-30 Thread Vladimir Yakovlev
I fixed typos and added CalangeLog. 2012-12-27 Vladimir Yakovlev : > On Sat, Dec 29, 2012 at 5:57 PM, Vladimir Yakovlev > wrote: >> I did changes. Please take a look. >> >> 2012/12/29, Uros Bizjak : >>> On Sat, Dec 29, 2012 at 6:26

Re: [RFC, x86] Changes for AVX and AVX2 processors

2013-01-10 Thread Vladimir Yakovlev
Hello Uros, It seems I didn't sent a patch with last changes. Sorry if so. Vladimir 2012-12-27 Vladimir Yakovlev : > On Sun, Dec 30, 2012 at 5:05 PM, Vladimir Yakovlev > wrote: >> I fixed typos and added CalangeLog. >> >> 2012-12-27 Vladimir Yakovlev > &

Re: [RFC, x86] Changes for AVX and AVX2 processors

2013-01-11 Thread Vladimir Yakovlev
I've fixed Changelog. Can we commit the patch to trunk now? 2012-12-27 Vladimir Yakovlev * config/i386/i386-c.c (ix86_target_macros_internal): New case. (ix86_target_macros_internal): Likewise. * config/i386/i386.c (m_CORE2I7): Removed. (m_CORE_HASWELL)

Re: [RFC, x86] Changes for AVX and AVX2 processors

2013-01-11 Thread Vladimir Yakovlev
I sent the patch. Send it once more. 2013/1/11 Jakub Jelinek : > On Fri, Jan 11, 2013 at 03:25:47PM +0400, Vladimir Yakovlev wrote: >> I've fixed Changelog. Can we commit the patch to trunk now? >> >> 2012-12-27 Vladimir Yakovlev >> >> * config/i386

Re: [RFC, x86] Changes for AVX and AVX2 processors

2013-01-11 Thread Vladimir Yakovlev
Kirill, Could you commit patch? 2013-01-11 Vladimir Yakovlev * config/i386/i386-c.c (ix86_target_macros_internal): New case. (ix86_target_macros_internal): Likewise. * config/i386/i386.c (m_CORE2I7): Removed. (m_CORE_HASWELL): New macro. (m_CORE_ALL

[PATCH] Changes in mode switching

2012-08-23 Thread Vladimir Yakovlev
e --prefix=/export/users/vbyakovl/workspaces/vzu/install-ref --enable-languages=c,c++,fortran --with-arch=corei7 --with-cpu=corei7 --with-fpmath=sse Ok for trank. 2012-08-25 Vladimir Yakovlev * mode-switching.c (transp): Changed type (make_preds_opaque): Added a

Re: [PATCH] Changes in mode switching

2012-08-24 Thread Vladimir Yakovlev
Thank you for testing. > With commenting out "if (i != mode)" of the hunk I changed type of transp and added this checking because if we reset transp[mode], then later in the loop FOR_EACH_BB (bb) sbitmap_not (kill[bb->index], transp[i][bb->index]); we set kill of the bb for that

Re: [PATCH] Changes in mode switching

2012-09-13 Thread Vladimir Yakovlev
Hello, I reproduced the failure and found reason of it. I understood haw it resolve and now I need small changes only - additional argument of EMIT_MODE_SET. Is it good fo trunk? Thank you, Vladimir 2012-09-14 Vladimir Yakovlev * (optimize_mode_switching): Added an argument

Re: [PATCH] Changes in mode switching

2012-09-14 Thread Vladimir Yakovlev
/configure --prefix=/export/users/vbyakovl/workspaces/vzu/install-middle --enable-languages=c,c++,fortran 2012/9/14 Vladimir Yakovlev : > Hello, > > I reproduced the failure and found reason of it. I understood haw it > resolve and now I need small changes only - additional argument of > EM

Re: [PATCH] Changes in mode switching

2012-09-17 Thread Vladimir Yakovlev
> Looks OK to me, though I have no authority to approve it > except SH specific part. Is there any more comments? Can it be committed in trank? Regards, Vladimir 2012/9/14 Kaz Kojima : > Vladimir Yakovlev wrote: >> I reproduced the failure and found reason of it. I understood ha

Re: [PATCH] Changes in mode switching

2012-09-18 Thread Vladimir Yakovlev
P.S. I'll be in vacation till end of nonth. > Vladimir Yakovlev writes: >> I reproduced the failure and found reason of it. I understood haw it >> resolve and now I need small changes only - additional argument of >> EMIT_MODE_SET. Is it good fo trunk? > > I'm

Re: [PATCH] Changes in mode switching

2012-09-18 Thread Vladimir Yakovlev
I tried to perform vzeroupper emitting after reload as additional pass of mode switching. I sow one problem that I don't know haw to overcome. After 'pro_and_epilogue', there can be no flow edge to exit block and pre_exit block is not created in this case (see rotine create_pre_exit). Without that

Fix performance regression with -mtune=atom

2011-09-30 Thread Vladimir Yakovlev
This patch fixes performance regression with -mtune=atom. Changing atom cost removes regression in several tests of EEMBC and spec2000. Bootstrap amd make check Ok for both with and witout -mtune-atom. OK for trunk? 2011-09-30 Yakovlev Vladimir vladimir.b.yakov...@intel.com * gcc/config/i

Re: [PATCH, Atom] Fix performance regression with -mtune=atom

2011-10-14 Thread Vladimir Yakovlev
This is a ping. Change affects Atom only and was made because it really gives better performance on this architecture. This fact actually leads to the thought that old value is just a simple misprint. Please look. Vladimir 2011/9/30 Vladimir Yakovlev : > This patch fixes performance regress

Re: [PATCH, Atom] Fix performance regression with -mtune=atom

2011-10-14 Thread Vladimir Yakovlev
Could anyone checkin that? Thanks, Vladimir 2011/10/14 Uros Bizjak : > Hello! > >> This is a ping. Change affects Atom only and was made because it >> really gives better performance on this architecture. This fact >> actually leads to the thought that old value is just a simple >> misprint. >> >

Re: [PATCH, testsuite] Fix for PR47440 - Use LCM for vzeroupper insertion

2011-07-20 Thread Vladimir Yakovlev
Hi Steven, I need a separate pass because the transformation on vzeroupper redandancy elemination must be performed when reload is completed. I think it is possible to make it as a separate pass in ix86-reorg if the phase works after reload. Thaks, Vladimir 2011/7/19 Steven Bosscher : >>        

Re: [PATCH, testsuite] Fix for PR47440 - Use LCM for vzeroupper insertion

2011-07-20 Thread Vladimir Yakovlev
Hi Uros, Thank you for such a detailed explanation. I'll study it. Regards, Vladimir 2011/7/20 Uros Bizjak : > Hello! > >> > ? ? ? ?* a/gcc/gcse.c (alloc_gcse_mem): Added code to run in PRE2. >> >> And this is necessary because...??? >> >> Why not just make it a separate pass in ix86-reorg that