This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] x86: {f,}xsave64 / {f,}xrstor64 / xsaveopt64 should not allow q suffix
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Jan Beulich <JBeulich at suse dot com>
- Cc: Binutils <binutils at sourceware dot org>
- Date: Fri, 10 Nov 2017 05:05:04 -0800
- Subject: Re: [PATCH] x86: {f,}xsave64 / {f,}xrstor64 / xsaveopt64 should not allow q suffix
- Authentication-results: sourceware.org; auth=none
- References: <5A05AA49020000780018DEB7@prv-mh.provo.novell.com>
On Fri, Nov 10, 2017 at 4:31 AM, Jan Beulich <JBeulich@suse.com> wrote:
> Just like is the case for xsave{s,c}64 and xrstors64 already. I wonder
> though why xsave{s,c} and xrstors don't allow for the q suffix, other
> than the other insns without the "64" suffix do.
commit eacc9c891d71cfef7f5c1f152291daeab785b3d4
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Fri Dec 4 07:51:41 2009 +0000
Support fxsave64 and fxrstor64.
gas/testsuite/
2009-12-03 H.J. Lu <hongjiu.lu@intel.com>
* gas/i386/i386.exp: Run x86-64-fxsave and x86-64-fxsave-intel.
* gas/i386/rex.d: Updated for fxsave64.
* gas/i386/x86-64-fxsave-intel.d: New.
* gas/i386/x86-64-fxsave.d: Likewise.
* gas/i386/x86-64-fxsave.s: Likewise.
were added after fxsaveq was introduced. Otherwise, fxsaveq won't have been
there.
> opcodes/
> 2017-11-10 Jan Beulich <jbeulich@suse.com>
>
> * i386-opc.tbl (fxsave64, fxrstor64, xsave64, xrstor64,
> xsaveopt64): Add No_qSuf.
> * i386-tbl.h: Re-generate.
>
OK. Thanks.
--
H.J.