> Please provide updated ChangeLog. --- gcc/ChangeLog (revision 206106) +++ gcc/ChangeLog (working copy) @@ -1,3 +1,14 @@ +2013-12-19 Ganesh Gopalasubramanian <ganesh.gopalasubraman...@amd.com> + + * config/i386/i386.c: Include cfgloop.h. + (ix86_loop_memcount): New function. + (ix86_loop_unroll_adjust): New function. + (TARGET_LOOP_UNROLL_ADJUST): Define. + * config/i386/i386.h + (TARGET_ADJUST_UNROLL): Define. + * config/i386/x86-tune.def + (X86_TUNE_ADJUST_UNROLL): Define. +
> The function comment is missing. Maybe you should also describe magic number > 32 here? Added the function comment. > Otherwise, the patch looks OK. Thanks. Bootstrapping passes. Is it OK for upstream? > BTW: Please avoid top-posting, see e.g. [1] for reasons... Sorry for the lapse. Will comply. Regards Ganesh
unroll-adjust.patch
Description: unroll-adjust.patch
