Hi, As suggested, I have created updated patch with diff -up after removing ChangeLog. If patch applies fine, please check it in. Regards, Saravanan -----Original Message----- From: H.J. Lu [mailto:hjl.tools@gmail.com] Sent: Friday, August 10, 2012 10:11 PM To: Ekanathan, Saravanan Cc: binutils@sourceware.org Subject: Re: [PATCH] Fix for PR 14423 - Assembler doesn't recognize fma instructions in bdver2 core On Thu, Aug 9, 2012 at 11:23 PM, Ekanathan, Saravanan wrote: > Thanks H.J.Lu, for the review comments. > As per the suggestion, I have re-used assembly files(arch-10.s and x86-64-arch-2.s) and created two new .d(objdump) files to do bdver2 specific testing. > If the patch is fine, please check it in on behalf of me. > Regards, Please submit patches with "diff -up" or you can use git instead of CVS: http://sourceware.org/git/?p=binutils.git;a=summary Please don't include ChangeLog entries in diff output if you need someone else to check in your patch for you. -- H.J.