This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Feature request: Set the direction flag on x86/x64 register->register operations


On Fri, Dec 19, 2008 at 7:25 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Fri, Dec 19, 2008 at 1:07 AM, Jan Beulich <jbeulich@novell.com> wrote:
>>>>> Stefan Dösinger<stefan@codeweavers.com> 19.12.08 01:52 >>>
>>>I had to regenerate opcodes/i386-tbl.h, but now it works like a charm!
>>>Objdump also disassembles the opcodes properly with the .r suffix
>>
>> I would strongly suggest to print to .r only with -Msuffix.
>
> Good point. I will do that.
>
>> Also, the whole thing seems half-baked to me, as it only deals with
>> alternative encodings using the 'D' bit. Using an .a suffix and handling
>
> ".r" handles insns which can be encoded differently by swapping operands
> in encoding:
>
>        fadd    %st,%st
>        fadd.r  %st,%st
>        fdiv    %st,%st
>
> I think  ".s" a better name. I will change ".r" to ".s".
>

Here is the updated patch. I changed ".r" to ".s". I also
removed FloatD since you can encode them differently
with %st vs %st(0).   Stefan, does it work for you?


-- 
H.J.

Attachment: binutils-option-3.patch
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]