On Wed, Dec 11, 2013 at 11:27 AM, Gopalasubramanian, Ganesh <ganesh.gopalasubraman...@amd.com> wrote:
> Accommodated the changes that you mentioned. > Completed the bootstrap testing too. Please provide updated ChangeLog. The function comment is missing. Maybe you should also describe magic number 32 here? +static unsigned +ix86_loop_unroll_adjust (unsigned nunroll, struct loop *loop) + if (!ix86_tune_features [X86_TUNE_ADJUST_UNROLL]) + return nunroll; Please add #define TARGET_ADJUST_UNROLL \ ix86_tune_features[X86_TUNE_ADJUST_UNROLL] to i386.h and use definition here. Otherwise, the patch looks OK. BTW: Please avoid top-posting, see e.g. [1] for reasons... [1] http://gcc.gnu.org/ml/gcc/2008-08/msg00133.html Uros.