This is the mail archive of the 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: [PATCH 3/6] x86: improve operand reversal

On Tue, Aug 7, 2018 at 12:37 AM, Jan Beulich <> wrote:
>>>> On 06.08.18 at 18:25, <> wrote:
>> On Mon, Aug 6, 2018 at 8:59 AM, Jan Beulich <> wrote:
>>>>>> On 06.08.18 at 17:09, <> wrote:
>>>> But there is no indication at all in your patch to show that it does
>>>> anything remotely to {load} nor {store}.   All your testcase changes
>>>> are for the ".s" suffix.
>>> There's a whole lot of stuff getting added to *pseudos.s.
>> That will make sure that {load} and {store} are handler properly
>> by actually testing them, instead of relying on the deprecated
>> ".s" suffix.
> I'm afraid I'm now confused by the "will" in your response: This
> makes it sound as if you assume something is yet to be added. But
> all those tests are already there. Bottom line - I'm still hanging in
> the air as to which way to proceed (see the earlier enumerated
> three options).

Your change is't needed unless you can show that it improves {load}
or {store}.


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