[PATCH] Add znver3 processor

Gopalasubramanian, Ganesh Ganesh.Gopalasubramanian@amd.com
Tue Oct 20 18:47:31 GMT 2020


[AMD Public Use]

Hi H.J,

> Please submit the updated patch so that one can apply with "git am".

PFA, the updated patch. Added "Word|Dword|Qword" for pvalidate. The test "snp.s" covers these operands in rAX.
Changelog is included in the patch. "git am" of the patch works.

Please approve and commit on my behalf.

Regards
Ganesh

-----Original Message-----
From: H.J. Lu <hjl.tools@gmail.com> 
Sent: Tuesday, October 20, 2020 9:23 PM
To: Jan Beulich <jbeulich@suse.com>
Cc: Gopalasubramanian, Ganesh <Ganesh.Gopalasubramanian@amd.com>; binutils@sourceware.org
Subject: Re: [PATCH] Add znver3 processor

[CAUTION: External Email]

On Tue, Oct 20, 2020 at 3:17 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> On 20.10.2020 11:55, Gopalasubramanian, Ganesh wrote:
> >> As far as the SNP insns go, aren't the templates lacking Cpu64 (and the disassembler then respective mode-dependent decoding)?
> > PVALIDATE is CPL0 executable in all operating modes. Apart from that, I have added all the instructions under Cpu64. Please review.
>
> Good point as far as PVALIDATE goes, but then its rAX operand also 
> needs to allow for %ax (i.e. Word) afaict.
>
> > If you approve the patch, can you please commit on my behalf.
>
> I'm okay with the patch with the above adjustment (and respective test 
> case addition), but I can't approve the patch. That'll be only H.J.
>

Please submit the updated patch so that one can apply with "git am".


--
H.J.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: BuZnver3.patch
Type: application/octet-stream
Size: 24267 bytes
Desc: BuZnver3.patch
URL: <https://sourceware.org/pipermail/binutils/attachments/20201020/46f80817/attachment-0001.obj>


More information about the Binutils mailing list