Thanks, Jakub! Updated Changelog:
gcc/ 2013-06-03 Yuri Rumyantsev <yuri.s.rumyant...@intel.com> * config/i386/i386.c (ix86_lea_outperforms): Fix formatting. (ix86_avoid_lea_for_addr): Likewise. (exact_dependency_1): Likewise. (ix86_adjust_cost): Likewise. (swap_top_of_ready_list): Fix formatting and !reload_completed check removed. (do_reorder_for_imul): Fix typo, formatting and !reload_completed check removed. (ix86_sched_reorder): Fix typo and formatting. (fold_builtin_cpu): Move M_INTEL_SLM at the end of processor types list. libgcc/ 2013-06-03 Yuri Rumyantsev <yuri.s.rumyant...@intel.com> * config/i386/cpuinfo.c (INTEL_SLM): New enum value. All testing is ok. Uros, is it ok for trunk? ------------ 2013/6/3 Jakub Jelinek <ja...@redhat.com>: > On Mon, Jun 03, 2013 at 05:00:38PM +0400, Yuri Rumyantsev wrote: >> Thanks a lot for you remarks! >> >> Attached is the patch which fixes all of them. >> >> Ok to install if all testing passes? >> >> Changelog: >> >> gcc/ >> >> 2013-06-03 Yuri Rumyantsev <yuri.s.rumyant...@intel.com> > > Two spaces before name, not just one. > >> * config/i386/i386.c (ix86_lea_outperforms): Fix formatting. > > Just one space after *, and tab indented CL lines. > Other than that it looks good to me, but I'll let Uros chime in. > > Thanks. > > Jakub