This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFC - ARM] - Fix PR43440 - Fix Neon inline asm register aliasing issues.
- From: Richard Henderson <rth at redhat dot com>
- To: Richard Earnshaw <rearnsha at arm dot com>
- Cc: ramana dot radhakrishnan at arm dot com, gcc-patches at gcc dot gnu dot org, rguenther at suse dot de
- Date: Wed, 27 Oct 2010 14:35:08 -0400
- Subject: Re: [RFC - ARM] - Fix PR43440 - Fix Neon inline asm register aliasing issues.
- References: <alpine.DEB.2.00.1003200039160.9269@liliput> <1269268484.20488.18.camel@e200601-lin.cambridge.arm.com> <1288193560.2025.8.camel@rwe-pc> <4CC86631.3060906@redhat.com> <1288202539.2898.2.camel@rwe-pc>
On 10/27/2010 02:02 PM, Richard Earnshaw wrote:
> On Wed, 2010-10-27 at 13:49 -0400, Richard Henderson wrote:
>> On 10/27/2010 11:32 AM, Richard Earnshaw wrote:
>>> PING - this needs a GWP review, I think.
>>
>> Pointer to the actual patch?
>
> http://gcc.gnu.org/ml/gcc-patches/2010-03/msg00978.html
>
>>
>> I had a look back at the original TARGET_MD_ASM_CLOBBERS
>> patch and thought it looked pretty good.
>>
>> I can't immediately think what good the O_R_N macro can
>> do outside the context of the asm clobbers. And if that
>> is the case, I can't imagine that this new macro is any
>> cleaner than the already clean T_M_A_C patch.
>>
>
> My patch doesn't have the side effects that Ramana originally
> mentioned. It also works correctly when -ffixed-<reg> and friends are
> used on the command line.
I see, yes.
Your patch is ok. Remember to update tm.texi.in now.
I don't think a target hook is really needed until we come up with
a way to handle the other register definition macros too. I think
the interface for register names, numbers, classes, etc probably
want a unified solution rather than simple-minded piecemeal changes.
r~