RE: [PATCH] Fix for PR 14423 - Assembler doesn't recognize fma instructions in bdver2 core

["Ekanathan, Saravanan" <Saravanan dot Ekanathan at amd dot com>]

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,
Saravanan
-----Original Message-----
From: H.J. Lu [mailto:hjl.tools@gmail.com] 
Sent: Thursday, August 09, 2012 6:24 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 1:14 AM, Ekanathan, Saravanan <Saravanan.Ekanathan@amd.com> wrote:
> Hi,
>   The files (arch-10.s and x86-64-arch-2.s)  already does fma testing but it doesn't test whether a specific core (bdver2) supports fma or not.
> Re-using these assembly files and creating new .d(objdump) files with -march=bdver2 will not help, because these files have un-supported bdver2 instructions.
>
> So, is there a way to test a core against one/all of the supported instructions in the existing Assembler test framework?
> If not, adding a new testscase for this bug is the only option I guess.
> Regards,
> Saravanan
>

You can create new *.d files, similar to arch-10-lzcnt.d/x86-64-arch-2-lzcnt.d, with

-march=bdver2+....


--
H.J.

Attachment: bdver2_fma.patch
Description: bdver2_fma.patch

Attachment: testsuite.patch
Description: testsuite.patch