Enable support to Intel Key locker instructions.
Cui, Lili
lili.cui@intel.com
Wed Sep 23 02:09:05 GMT 2020
> On Tue, Sep 22, 2020 at 9:08 AM Jan Beulich <jbeulich@suse.com> wrote:
> > >> ... in this table you'll notice that MOD_0F38* all go together, and
> > >> _later_ there's a MOD_VEX_0F38* group. I notice that recent
> > >> additions (of yours?) also already violate this sorting model - please may
> I ask for this to corrected as well?
> > >> The more outliers we have there, the more difficult will it be to
> > >> maintain this code.
> > >
> > > Thank you reviewing my patch. I put MOD_VEX_0F38* together.
> >
> > Imo this should be a separate change, not merged into here.
>
> Yes, please make a separate patch.
>
Ok, I will fix it with a separate patch, thanks.
> > >> ... these four need special treatment in output_insn()'s setting of
> > >> GNU_PROPERTY_X86_FEATURE_2_XMM, due to the lack of explicit
> RegXMM
> > >> operands.
> > >>
> > > Added it.
> >
> > If I was making a change like this, I'm pretty sure H.J. would ask me
> > to also add test cases for it.
> >
>
> Yes, please add a new testcase. See:
>
> commit a7e12755d57879884c523cae1cf009efc9da933c
> Author: H.J. Lu <hjl.tools@gmail.com>
> Date: Wed Feb 19 04:54:45 2020 -0800
>
> x86: Mark cvtpi2ps and cvtpi2pd as MMX
>
> for an example.
>
> Thanks.
>
> --
> H.J.
Added it, thanks.
Lili.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Enable-support-to-Intel-Keylocker-instructions.patch
Type: application/octet-stream
Size: 26740 bytes
Desc: 0001-Enable-support-to-Intel-Keylocker-instructions.patch
URL: <https://sourceware.org/pipermail/binutils/attachments/20200923/601a2df6/attachment-0001.obj>
More information about the Binutils
mailing list