[PATCH, v3]: AMD btver1 and btver2 enablement

[Eggone, NagaJyothi]

> -----Original Message-----
> From: H.J. Lu [mailto:hjl.tools@gmail.com]
> Sent: Friday, August 17, 2012 6:41 PM
> To: Eggone, NagaJyothi
> Cc: binutils@sourceware.org
> Subject: Re: [PATCH, v3]: AMD btver1 and btver2 enablement
> 
> On Thu, Aug 16, 2012 at 11:28 PM, Eggone, NagaJyothi
> <NagaJyothi.Eggone@amd.com> wrote:
> > Hello H.J.,
> >
> >> -----Original Message-----
> >> From: H.J. Lu [mailto:hjl.tools@gmail.com]
> >> Sent: Thursday, August 16, 2012 11:31 PM
> >> To: Eggone, NagaJyothi
> >> Cc: binutils@sourceware.org
> >> Subject: Re: [PATCH, v2]: AMD btver1 and btver2 enablement
> >>
> >> >>
> >> >> Do you really need Cpu3dnowPrfch? Where will it be used?
> >> >
> >> > The 3dnow 'prefetch' and 'prefetchw' instructions are available
> both
> >> on the btver1 and btver2 cores. I could not have enabled both these
> >> instructions for these cores without adding a cpu flag bit (this
> >> corresponds to the cpuid ECX bit 8 on AMD processors).
> >>
> >> Please drop Cpu3dnowPrfch and add CpuPRFCHW to
> >> prefetch.
> >>
> >> gas/testsuite changes should be in gas/testsuite/ChangeLog.
> >>
> >> --
> >> H.J.
> >
> > I have changed the patch as per suggestions above. This patch also
> includes the diffs of files opcodes/i386-init.h and opcodes/i386-tbl.h.
> > Please review and if OK, commit to trunk.
> >
> 
> Please reformat your ChangeLog entries to follow
> existing ones.  You have extra blank lines and
> missing '.'.
> 
> +  /* Supports prefetchw and 3dnow prefetch instructions.  */
> 
> Please drop 3dnow here.
> 
> --
> H.J.

Re-submitting with changes.

- Nagajyothi

-------------- next part --------------
A non-text attachment was scrubbed...
Name: btver.patch
Type: application/octet-stream
Size: 8842 bytes
Desc: btver.patch
URL: <https://sourceware.org/pipermail/binutils/attachments/20120817/897cfda1/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: btver_gastest.patch
Type: application/octet-stream
Size: 34078 bytes
Desc: btver_gastest.patch
URL: <https://sourceware.org/pipermail/binutils/attachments/20120817/897cfda1/attachment-0001.obj>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: gas-changelog.txt
URL: <https://sourceware.org/pipermail/binutils/attachments/20120817/897cfda1/attachment.txt>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: gastest-changelog.txt
URL: <https://sourceware.org/pipermail/binutils/attachments/20120817/897cfda1/attachment-0001.txt>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: opcodes-changelog.txt
URL: <https://sourceware.org/pipermail/binutils/attachments/20120817/897cfda1/attachment-0002.txt>