This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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]

[PATCH] Fix PR target/31568 ICE with invalid %y operand (inline-asm)

From: "Andrew Pinski" <Andrew_Pinski at playstation dot sony dot com>
To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, "David Edelsohn" <dje at watson dot ibm dot com>
Date: Mon, 14 Jul 2008 14:43:32 -0700
Subject: [PATCH] Fix PR target/31568 ICE with invalid %y operand (inline-asm)

Hi,
  The problem here is that we use gcc_assert instead of calling
output_operand_lossage if the user tries to use %yN invalidly.  This
fixes the ICE and adds a testcase so we don't regress.

OK?  Bootstrapped and tested on powerpc64-linux-gnu with no
regressions and it has been in the PS3 compiler for about a year now.

Thanks,
Andrew Pinski

ChangeLog:
* config/rs6000/rs6000.c (print_operand <case 'y'>): Don't use
gcc_assert, instead call output_operand_lossage.



* gcc.target/powerpc/asm-y.c: New testcase.

Attachment: t.diff.txt
Description: Text document

Follow-Ups:
- Re: [PATCH] Fix PR target/31568 ICE with invalid %y operand (inline-asm)
  - From: David Edelsohn

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