This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH] Allow case-insensitive comparisons of register names by implementing CASE_INSENSITIVE_REGISTER_NAMES PR target/70320
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Jozef Lawrynowicz <jozef dot l at mittosystems dot com>
- Cc: Richard Sandiford <richard dot sandiford at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, segher at kernel dot crashing dot org
- Date: Fri, 19 Jul 2019 12:24:57 +0200
- Subject: Re: [PATCH] Allow case-insensitive comparisons of register names by implementing CASE_INSENSITIVE_REGISTER_NAMES PR target/70320
- References: <20190718204538.63c2260f@jozef-kubuntu> <email@example.com> <20190719103952.02de67cf@jozef-kubuntu> <20190719095443.GU2125@tucnak> <20190719111751.3ba0f927@jozef-kubuntu>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Jul 19, 2019 at 11:17:51AM +0100, Jozef Lawrynowicz wrote:
> That is something I considered in previous discussion here:
> But it seemed like this could potentially be a useful feature for other targets
> that wish to enable it. It doesn't appear necessary to restrict the case of
> register names, unless a target exists that has different registers that differ
> only by case.
It doesn't seem like a generally useful feature to me, C as well as C++ are
case sensitive, so is gcc command line parsing, so having the register names
handled insensitive is strange and undesirable.
Perhaps the reason you want it for msp430 is that the register names were
chosen badly as upper case which surprises people?
Having register int x __asm ("eAx"); register __m512i __asm ("ZmM11"); is simply
weird, not something we should allow nor promote.