Committed: Only run x86 space1 test for x86 targets

View: New views
1 Messages — Rating Filter:   Alert me  

Committed: Only run x86 space1 test for x86 targets

by Nick Clifton :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi Guys,

  I am applying the patch below to move H.J.'s newly created space1
  test to inside a check for an x86 target - it was creating a new
  failure in the gas testsuite for non-x86 targets.

Cheers
  Nick

gas/testsuite/ChangeLog
2009-11-05  Nick Clifton  <nickc@...>

        * gas/i386/i386.exp (space1): Move test inside check for x86
        target.

Index: gas/testsuite/gas/i386/i386.exp
===================================================================
RCS file: /cvs/src/src/gas/testsuite/gas/i386/i386.exp,v
retrieving revision 1.123
diff -c -3 -p -r1.123 i386.exp
*** gas/testsuite/gas/i386/i386.exp 4 Nov 2009 18:52:03 -0000 1.123
--- gas/testsuite/gas/i386/i386.exp 5 Nov 2009 10:57:54 -0000
*************** proc gas_32_check { } {
*** 17,25 ****
      return [regexp "targets:.*i386" [lindex $status 1]];
  }
 
- # Common tests
- run_list_test "space1" "-al"
-
  if [expr ([istarget "i*86-*-*"] ||  [istarget "x86_64-*-*"]) && [gas_32_check]]  then {
 
      global ASFLAGS
--- 17,22 ----
*************** if [expr ([istarget "i*86-*-*"] ||  [ist
*** 208,217 ****
--- 205,216 ----
      set ASFLAGS "$old_ASFLAGS"
  }
 
+ # Common tests
  if [expr [istarget "i*86-*-*"] || [istarget "x86_64-*-*"]] then {
      run_dump_test "intel-expr"
      run_dump_test "string-ok"
      run_list_test "string-bad" ""
+     run_list_test "space1" "-al"
  }
 
  if [expr ([istarget "i*86-*-*"] || [istarget "x86_64-*-*"]) && [gas_64_check]] then {