x86: Support Intel UINTR

H.J. Lu hjl.tools@gmail.com
Tue Oct 13 11:36:01 GMT 2020


On Tue, Oct 13, 2020 at 2:33 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> On 13.10.2020 11:00, Cui, Lili wrote:
> >> On 13.10.2020 08:11, Cui, Lili wrote:
> >>> --- a/gas/testsuite/gas/i386/prefix.s
> >>> +++ b/gas/testsuite/gas/i386/prefix.s
> >>> @@ -367,14 +367,6 @@
> >>>
> >>>     nop
> >>>
> >>> -# repz; rdrand %eax
> >>> -   .byte 0xf3
> >>> -   .byte 0x0f
> >>> -   .byte 0xc7
> >>> -   .byte 0xf0
> >>> -
> >>> -   nop
> >>> -
> >>
> >> Instead of outright dropping this, can you see about finding a suitable
> >> replacement encoding to retain what's being tested here?
> >
> > Thanks for your review, I tried but didn't find a replaceable encoding here,
> > as RDRAND not support F2/F3 prefixes, so H.J added an invalid test case(repz rdrand %eax) here,
> > "repz rdrand" encoding is same with "senduipi",  so I have to delete this testcase.
> >
> >
> > Rdrand  NFx REX.W + 0F C7 /6
> >
> > NFx ? Indicates the use of F2/F3 prefixes (beyond those already part of the instructions opcode) are not
> > allowed with the instruction. Such use will either cause an invalid-opcode exception (#UD) or result in the
> > encoding for a different instruction.
>
> Question is whether HJ really mean to test the RDRAND major opcode
> specifically, or whether some other one could be use in its stead.
> HJ?
>
> Jan

Please drop

        * testsuite/gas/i386/prefix.d: There is a test
        case binary conflict with senduipi opcode delete it.
        * testsuite/gas/i386/prefix.s: Likewise.

since they are no longer needed.

-- 
H.J.


More information about the Binutils mailing list