This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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]

ppc_symbol_chars


I'm not sure why '+' and '-' were added the ppc_symbol_chars.  The
comment referring to branches isn't correct.  Maybe it was at some
stage, but insn mnemonics aren't operands, so are not affected by
what is accepted as a symbol.

Removing them cures the following

 .text
 lwz 3,x@l +4(9)

Error: syntax error; found ` ' but expected `('
Error: junk at end of line: `+4(9)'

	* config/tc-ppc.c (ppc_symbol_chars): Remove '+' and '-'.

Index: gas/config/tc-ppc.c
===================================================================
RCS file: /cvs/src/src/gas/config/tc-ppc.c,v
retrieving revision 1.113
diff -u -p -r1.113 tc-ppc.c
--- gas/config/tc-ppc.c	22 Sep 2006 11:05:27 -0000	1.113
+++ gas/config/tc-ppc.c	22 Sep 2006 13:36:13 -0000
@@ -182,11 +182,9 @@ const char EXP_CHARS[] = "eE";
    as in 0d1.0.  */
 const char FLT_CHARS[] = "dD";
 
-/* '+' and '-' can be used as postfix predicate predictors for conditional
-   branches.  So they need to be accepted as symbol characters.
-   Also, anything that can start an operand needs to be mentioned here,
+/* Anything that can start an operand needs to be mentioned here,
    to stop the input scrubber eating whitespace.  */
-const char ppc_symbol_chars[] = "+-%[";
+const char ppc_symbol_chars[] = "%[";
 
 /* The dwarf2 data alignment, adjusted for 32 or 64 bit.  */
 int ppc_cie_data_alignment;

-- 
Alan Modra
IBM OzLabs - Linux Technology Centre


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