Re: [PATCH] Perform case-insensitive comparison when decoding register names (PR target/70320)

2019-07-10 Thread Segher Boessenkool
On Wed, Jul 10, 2019 at 05:23:28PM +0100, Jozef Lawrynowicz wrote: > On Tue, 9 Jul 2019 16:36:46 -0500 > Segher Boessenkool wrote: > > But it is data, not a constant, so it does not allow optimising based > > on its potentially constant value? Where "potentially" in this case > > means "always" :

Re: [PATCH] Perform case-insensitive comparison when decoding register names (PR target/70320)

2019-07-10 Thread Jozef Lawrynowicz
On Tue, 9 Jul 2019 16:36:46 -0500 Segher Boessenkool wrote: > On Tue, Jul 09, 2019 at 10:16:31PM +0100, Jozef Lawrynowicz wrote: > > On Mon, 8 Jul 2019 16:42:15 -0500 > > Segher Boessenkool wrote: > > > > > > Ok, yes a DEFHOOKPOD or similar sounds like a good idea, I'll look into > > > > thi

Re: [PATCH] Perform case-insensitive comparison when decoding register names (PR target/70320)

2019-07-09 Thread Segher Boessenkool
On Tue, Jul 09, 2019 at 10:16:31PM +0100, Jozef Lawrynowicz wrote: > On Mon, 8 Jul 2019 16:42:15 -0500 > Segher Boessenkool wrote: > > > > Ok, yes a DEFHOOKPOD or similar sounds like a good idea, I'll look into > > > this > > > alternative. > > > > What is that, like target macros? But with

Re: [PATCH] Perform case-insensitive comparison when decoding register names (PR target/70320)

2019-07-09 Thread Jozef Lawrynowicz
On Mon, 8 Jul 2019 16:42:15 -0500 Segher Boessenkool wrote: > > Ok, yes a DEFHOOKPOD or similar sounds like a good idea, I'll look into this > > alternative. > > What is that, like target macros? But with some indirection? Yes its for target macros, it looks like the "POD" in DEFHOOKPOD stan

Re: [PATCH] Perform case-insensitive comparison when decoding register names (PR target/70320)

2019-07-08 Thread Segher Boessenkool
On Mon, Jul 08, 2019 at 10:21:29PM +0100, Jozef Lawrynowicz wrote: > On Mon, 08 Jul 2019 21:14:36 +0100 > Richard Sandiford wrote: > > > Segher Boessenkool writes: > > > It isn't obviously safe either. Are there any targets that have names > > > for different registers that differ only in case?

Re: [PATCH] Perform case-insensitive comparison when decoding register names (PR target/70320)

2019-07-08 Thread Jozef Lawrynowicz
On Mon, 08 Jul 2019 21:14:36 +0100 Richard Sandiford wrote: > Segher Boessenkool writes: > > On Thu, Jul 04, 2019 at 01:32:59PM +0100, Jozef Lawrynowicz wrote: > >> The attached patch allows the case of register names used in an asm > >> statement > >> clobber list to be disregarded when chec

Re: [PATCH] Perform case-insensitive comparison when decoding register names (PR target/70320)

2019-07-08 Thread Richard Sandiford
Segher Boessenkool writes: > On Thu, Jul 04, 2019 at 01:32:59PM +0100, Jozef Lawrynowicz wrote: >> The attached patch allows the case of register names used in an asm statement >> clobber list to be disregarded when checking the validity of the register >> names. >> >> Currently, the register na

Re: [PATCH] Perform case-insensitive comparison when decoding register names (PR target/70320)

2019-07-04 Thread Segher Boessenkool
On Thu, Jul 04, 2019 at 01:32:59PM +0100, Jozef Lawrynowicz wrote: > The attached patch allows the case of register names used in an asm statement > clobber list to be disregarded when checking the validity of the register > names. > > Currently, the register name used in asm statement clobber li

[PATCH] Perform case-insensitive comparison when decoding register names (PR target/70320)

2019-07-04 Thread Jozef Lawrynowicz
The attached patch allows the case of register names used in an asm statement clobber list to be disregarded when checking the validity of the register names. Currently, the register name used in asm statement clobber list must exactly match those defined in the targets REGISTER_NAMES macro. This