This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fail on invalid operand code in mips_print_operand
- From: Adam Nemet <anemet at caviumnetworks dot com>
- To: Richard Sandiford <rdsandiford at googlemail dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 12 May 2009 14:34:28 -0700
- Subject: Re: [PATCH] Fail on invalid operand code in mips_print_operand
- References: <18953.36435.163645.726315@ropi.home> <87my9injtv.fsf@talisman.home>
Richard Sandiford writes:
> I'm just a little bit worried about rejecting %z for memory operands.
> %z seems at most redundant in that case, and I can imagine some inline
> asms accidentally use %z for things that can be memories.
>
> What do you think? I don't have a strong opinion, just though I'd
> raise the question.
I agree. We don't gain much by rejecting %z on a mem operand since it can't
really change the meaning of the operand.
> (A version that allows '%z' for memories is preapproved though.)
Thanks. I retest with that and commit.
Adam