PR gas/14887
authorNick Clifton <nickc@redhat.com>
Mon, 7 Jan 2013 12:49:12 +0000 (12:49 +0000)
committerNick Clifton <nickc@redhat.com>
Mon, 7 Jan 2013 12:49:12 +0000 (12:49 +0000)
* config/tc-arm.c (skip_past_char): Skip whitespace before the
anticipated character.
* config/tc-arm.c (parse_address_main): Delete skip of whitespace
here as it is no longer needed.

PR gas/14887
* gas/arm/neon-ldst-es.s: Add more whitespace.

gas/ChangeLog
gas/config/tc-arm.c
gas/testsuite/ChangeLog
gas/testsuite/gas/arm/neon-ldst-es.s

index 73d1b93240dc39f0d8a4dc74a23a6fb7f0641d02..28c509572f2ff66668c4e8ad94d456a9319f2325 100644 (file)
@@ -1,3 +1,11 @@
+2013-01-07  Nick Clifton  <nickc@redhat.com>
+
+       PR gas/14887
+       * config/tc-arm.c (skip_past_char): Skip whitespace before the
+       anticipated character.
+       * config/tc-arm.c (parse_address_main): Delete skip of whitespace
+       here as it is no longer needed.
+
 2013-01-06  Andreas Schwab  <schwab@linux-m68k.org>
 
        * doc/c-mips.texi (MIPS Opts): Fix use of @itemx.
index 33eacdb5bd5e2b0f6422276b19c91e61305bdced..8cf5a8027090a4a3789f8f3273daef5839f3f260 100644 (file)
@@ -1,6 +1,6 @@
 /* tc-arm.c -- Assemble for the ARM
    Copyright 1994, 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003,
-   2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012
+   2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013
    Free Software Foundation, Inc.
    Contributed by Richard Earnshaw (rwe@pegasus.esprit.ec.org)
        Modified by David Taylor (dtaylor@armltd.co.uk)
@@ -885,6 +885,9 @@ const char FLT_CHARS[] = "rRsSfFdDxXeEpP";
 static inline int
 skip_past_char (char ** str, char c)
 {
+  /* PR gas/14987: Allow for whitespace before the expected character.  */
+  skip_whitespace (*str);
+  
   if (**str == c)
     {
       (*str)++;
@@ -5168,6 +5171,9 @@ parse_address_main (char **str, int i, int group_relocations,
       return PARSE_OPERAND_SUCCESS;
     }
 
+  /* PR gas/14887: Allow for whitespace after the opening bracket.  */
+  skip_whitespace (p);
+
   if ((reg = arm_reg_parse (&p, REG_TYPE_RN)) == FAIL)
     {
       inst.error = _(reg_expected_msgs[REG_TYPE_RN]);
@@ -5291,9 +5297,6 @@ parse_address_main (char **str, int i, int group_relocations,
        return result;
     }
 
-  /* PR gas/14987: Allow for whitespace before the closing bracket.  */
-  skip_whitespace (p);
-
   if (skip_past_char (&p, ']') == FAIL)
     {
       inst.error = _("']' expected");
index 0cb28564d0f7165b05ba18eb812e1aaf3404c8c4..c27a794aabb3ff1a15381d991bfb5d5380dda9a3 100644 (file)
@@ -1,3 +1,8 @@
+2013-01-07  Nick Clifton  <nickc@redhat.com>
+
+       PR gas/14887
+       * gas/arm/neon-ldst-es.s: Add more whitespace.
+
 2013-01-04  Juergen Urban <JuergenUrban@gmx.de>
 
        * gas/mips/r5900-full.s: New test.
index ea7de078ee8b4de804ec8ad5457c311a49e87e2f..79f4b0cdd3de1fbed52e844a928f0f497b0619df 100644 (file)
@@ -58,5 +58,5 @@
        vld4.32 {q1-q2},[r7]
        vld4.32 {q14-q15},[r7]
 
-       @ PR 14987: Allow for whitespace before the closing bracket.
-       vld1.32 {d1[]}, [r2 ], r3
+       @ PR 14987 and 14887: Allow for whitespace in the instruction.
+       vld1.32 { d1 [ ] } , [ r2 ] , r3