fix some gdb.dwarf2 tests for parallel safety
authorTom Tromey <tromey@redhat.com>
Thu, 22 Aug 2013 13:39:13 +0000 (13:39 +0000)
committerTom Tromey <tromey@redhat.com>
Thu, 22 Aug 2013 13:39:13 +0000 (13:39 +0000)
This fixes a few gdb.dwarf2 tests to be more parallel-safe.  This
mostly amounts to changing them to write their files into the
directory designated by standard_output_file.

Built and regtested on x86-64 Fedora 18.

* gdb.dwarf2/clztest.exp: Use standard_testfile.
* gdb.dwarf2/dw2-minsym-in-cu.exp: Use standard_testfile.
* gdb.dwarf2/fission-base.S: Remove directory from
DW_AT_GNU_dwo_name.
* gdb.dwarf2/fission-base.exp: Use build_executable.  Set
debug-file-directory.
* gdb.dwarf2/fission-reread.S: Remove directory from
DW_AT_GNU_dwo_name.
* gdb.dwarf2/fission-reread.exp: Use build_executable.  Set
debug-file-directory.

gdb/testsuite/ChangeLog
gdb/testsuite/gdb.dwarf2/clztest.exp
gdb/testsuite/gdb.dwarf2/dw2-minsym-in-cu.exp
gdb/testsuite/gdb.dwarf2/fission-base.S
gdb/testsuite/gdb.dwarf2/fission-base.exp
gdb/testsuite/gdb.dwarf2/fission-reread.S
gdb/testsuite/gdb.dwarf2/fission-reread.exp

index 661042e340bbe6895bf83388a1e45959e49e05bd..f32933885d007a838e368b3c912b12ea53d3bb0b 100644 (file)
@@ -1,3 +1,16 @@
+2013-08-22  Tom Tromey  <tromey@redhat.com>
+
+       * gdb.dwarf2/clztest.exp: Use standard_testfile.
+       * gdb.dwarf2/dw2-minsym-in-cu.exp: Use standard_testfile.
+       * gdb.dwarf2/fission-base.S: Remove directory from
+       DW_AT_GNU_dwo_name.
+       * gdb.dwarf2/fission-base.exp: Use build_executable.  Set
+       debug-file-directory.
+       * gdb.dwarf2/fission-reread.S: Remove directory from
+       DW_AT_GNU_dwo_name.
+       * gdb.dwarf2/fission-reread.exp: Use build_executable.  Set
+       debug-file-directory.
+
 2013-08-16  Doug Evans  <dje@google.com>
 
        * lib/prelink-support.exp (prelink_yes): Flag test as unsupported if
index 127fbdfd87ac71dfb90a4fd5099c5f4ed866a41c..0874342dab8d0c84ec52dd6c47fe7cf571a596a9 100644 (file)
@@ -15,6 +15,7 @@
 
 load_lib dwarf.exp
 
+standard_testfile .S
 set test "clztest"
 
 # This test can only be run on targets which support DWARF-2 and use gas.
index 105123f62f92f3456c6d8f2591b764dbcb93021a..9b2a8054bfa99cfaffc7fe1589596851a5c2ddce 100644 (file)
@@ -21,11 +21,9 @@ if {![dwarf2_support]} {
 
 # This testfile has reproducibility only with cc-with-index.sh.
 
-set testfile "dw2-minsym-in-cu"
-set srcfile ${testfile}.S
-set executable ${testfile}
+standard_testfile .S
 
-if [prepare_for_testing ${testfile}.exp ${executable} ${srcfile}] {
+if [prepare_for_testing ${testfile}.exp ${testfile} ${srcfile}] {
     return -1
 }
 
index 23de5a7131e885399785e6709c6bd4ceb2547491..14b2494495cc1e03d18a562984b55844580bfa65 100644 (file)
@@ -145,10 +145,7 @@ main:
        .ascii "/tmp/src/gdb/testsuite\0"       # DW_AT_comp_dir
        # Normally dwo_name would be "fission-base.dwo".
        # Simplification: Leave the DWO contents in the executable.
-       # "gdb.dwarf2/" is included so that gdb can load the file without
-       # having to set "debug-file-directory".  This is useful when
-       # testing with cc-with-tweaks.sh -i.
-       .ascii "gdb.dwarf2/fission-base\0"      # DW_AT_GNU_dwo_name
+       .ascii "fission-base\0" # DW_AT_GNU_dwo_name
        .4byte  .Ldebug_pubnames0       # DW_AT_GNU_pubnames
        .4byte  .Ldebug_pubtypes0       # DW_AT_GNU_pubtypes
        .4byte  .Ldebug_addr0_begin     # DW_AT_GNU_addr_base
index 88e4ff1a3eaccc85b71b9692d5212a53f1b430e7..58448f8b5189f2e815139193fd550bcb2ac01943 100644 (file)
@@ -29,10 +29,19 @@ set basename "fission-base"
 
 standard_testfile .S
 
-if { [prepare_for_testing "$testfile.exp" "$testfile" "$srcfile" {nodebug}] } {
+if { [build_executable "$testfile.exp" "$testfile" "$srcfile" {nodebug}] } {
     return -1
 }
 
+gdb_exit
+gdb_start
+gdb_reinitialize_dir $srcdir/$subdir
+
+gdb_test_no_output "set debug-file-directory [file dirname $binfile]" \
+    "set debug-file-directory"
+
+gdb_load ${binfile}
+
 if ![runto_main] {
     return -1
 }
index c59882406577483e10d313362fa78fea3269a7a3..e566cc6a45a758c1bd76374fa929ad97c7bf4f18 100644 (file)
@@ -135,11 +135,8 @@ main:
        .uleb128 0x2    /* (DIE (0) DW_TAG_type_unit) */
        .ascii "/tmp/src/gdb/testsuite\0"       /* DW_AT_comp_dir */
        /* Normally dwo_name would be "fission-reread.dwo".
-          Simplification: Leave the DWO contents in the executable.
-          "gdb.dwarf2/" is included so that gdb can load the file without
-          having to set "debug-file-directory".  This is useful when
-          testing with cc-with-tweaks.sh -i.  */
-       .ascii "gdb.dwarf2/fission-reread\0"    /* DW_AT_GNU_dwo_name */
+          Simplification: Leave the DWO contents in the executable.  */
+       .ascii "fission-reread\0"       /* DW_AT_GNU_dwo_name */
        .4byte  .Ldebug_pubnames0       /* DW_AT_GNU_pubnames */
        .4byte  .Ldebug_pubtypes0       /* DW_AT_GNU_pubtypes */
        .4byte  .Ldebug_addr0   /* DW_AT_GNU_addr_base */
index 0f06326f2edcfe70d9d5930f9bb984bb935ccc68..f275308d47334e7b051754534b2dee937300839e 100644 (file)
@@ -24,10 +24,19 @@ set basename "fission-reread"
 
 standard_testfile .S
 
-if { [prepare_for_testing "$testfile.exp" "$testfile" "$srcfile" {nodebug}] } {
+if { [build_executable "$testfile.exp" "$testfile" "$srcfile" {nodebug}] } {
     return -1
 }
 
+gdb_exit
+gdb_start
+gdb_reinitialize_dir $srcdir/$subdir
+
+gdb_test_no_output "set debug-file-directory [file dirname $binfile]" \
+    "set debug-file-directory"
+
+gdb_load ${binfile}
+
 gdb_test "break main" "Breakpoint.*at.*"
 
 # If we get this far gdb didn't crash, nor did an error occur.