This patch looks fine to me. 2011/6/6 Ramanarayanan, Ramshankar <ramshankar.ramanaraya...@amd.com>: > I would like a code review for the attached diff. > > > > This diff enables more control to X8664 “Scale, Index, Base” (SIB) flag > CG_merge_counters_x86 with the BOOL variable CG_merge_counters_x86_set. > Using this hook, we proceed to set CG_merge_counters_x86 at O3 thereby > enabling default SIB at O3. > > > > This change has been tested with SPEC 2006 and other FP applications. > > > > Best regards, > > Ram > > > > Member of Technical Staff, > > Open Source Compiler Engineering, > > Advanced Micro Devices > > ------------------------------------------------------------------------------ > Simplify data backup and recovery for your virtual environment with vRanger. > Installation's a snap, and flexible recovery options mean your data is safe, > secure and there when you need it. Discover what all the cheering's about. > Get your free trial download today. > http://p.sf.net/sfu/quest-dev2dev2 > _______________________________________________ > Open64-devel mailing list > Open64-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/open64-devel > >
-- Regards, Lai Jian-Xin ------------------------------------------------------------------------------ All of the data generated in your IT infrastructure is seriously valuable. Why? It contains a definitive record of application performance, security threats, fraudulent activity, and more. Splunk takes this data and makes sense of it. IT sense. And common sense. http://p.sf.net/sfu/splunk-d2d-c2 _______________________________________________ Open64-devel mailing list Open64-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/open64-devel
