Patchwork rs6000: Don't allow %. in PRINT_OPERAND_PUNCT_VALID_P

login
register
mail settings
Submitter Segher Boessenkool
Date Oct. 9, 2012, 10:53 p.m.
Message ID <191fcc086904ed9d2effdf5eb749fede75345a90.1349822959.git.segher@kernel.crashing.org>
Download mbox | patch
Permalink /patch/190508/
State New
Headers show

Comments

Segher Boessenkool - Oct. 9, 2012, 10:53 p.m.
As David noticed, I forgot PRINT_OPERAND_PUNCT_VALID_P in the patch
that removed %.  This fixes it.

Bootstrapped and regression tested on powerpc64-linux.  Okay to
apply?


Segher


2012-10-09  Segher Boessenkool  <segher@kernel.crashing.org>

gcc/
	* config/rs6000/rs6000.h (PRINT_OPERAND_PUNCT_VALID_P):
	Delete '.'.

---
 gcc/config/rs6000/rs6000.h |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)
David Edelsohn - Oct. 10, 2012, 1:13 a.m.
On Tue, Oct 9, 2012 at 6:53 PM, Segher Boessenkool
<segher@kernel.crashing.org> wrote:
> As David noticed, I forgot PRINT_OPERAND_PUNCT_VALID_P in the patch
> that removed %.  This fixes it.
>
> Bootstrapped and regression tested on powerpc64-linux.  Okay to
> apply?

> 2012-10-09  Segher Boessenkool  <segher@kernel.crashing.org>
>
> gcc/
>         * config/rs6000/rs6000.h (PRINT_OPERAND_PUNCT_VALID_P):
>         Delete '.'.

Okay.

Thanks, David

Patch

diff --git a/gcc/config/rs6000/rs6000.h b/gcc/config/rs6000/rs6000.h
index b968802..82388d9 100644
--- a/gcc/config/rs6000/rs6000.h
+++ b/gcc/config/rs6000/rs6000.h
@@ -2229,8 +2229,7 @@  extern char rs6000_reg_names[][8];	/* register names (0 vs. %r0).  */
 
 /* Define which CODE values are valid.  */
 
-#define PRINT_OPERAND_PUNCT_VALID_P(CODE)  \
-  ((CODE) == '.' || (CODE) == '&')
+#define PRINT_OPERAND_PUNCT_VALID_P(CODE)  ((CODE) == '&')
 
 /* Print a memory address as an operand to reference that memory location.  */