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: [PATCH] PowerPC improve support for Gekko & Broadway

From: "A.W. Chadwick" <Alex dot Chadwick at cl dot cam dot ac dot uk>
To: Alan Modra <amodra at gmail dot com>
Cc: binutils at sourceware dot org
Date: Thu, 26 Jul 2018 08:52:29 +0100
Subject: Re: [PATCH] PowerPC improve support for Gekko & Broadway
References: <d20679a0-7571-adc1-7a5a-76b2951f51ed@cl.cam.ac.uk> <20180724025444.GN3504@bubble.grove.modra.org>

On 2018-07-24 03:54, Alan Modra wrote:

The way other disjoint fields are handled is with PPC_OPSHIFT_INV in
powerpc_operand.shift.

I missed PPC_OPSHIFT_INV, that does indeed seem a better choice.

I also don't see the necessity to disable any of the existing
extended mtspr/mfspr insns

Fair.

Revised patch as follows.  Please verify I haven't accidentally broken
anything for you, and I'll then commit.

Many thanks, works for me.

Note that your patch as posted wouldn't apply due to mangled
whitespace.

Sorry for that, I guess I need to find a better email client!

References:
- [PATCH] PowerPC improve support for Gekko & Broadway
  - From: Alex Chadwick
- Re: [PATCH] PowerPC improve support for Gekko & Broadway
  - From: Alan Modra

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