gdb/testsuite: Add "get_endianness" convenience proc
authorAndreas Arnez <arnez@linux.vnet.ibm.com>
Tue, 13 Jun 2017 13:20:26 +0000 (15:20 +0200)
committerAndreas Arnez <arnez@linux.vnet.ibm.com>
Tue, 13 Jun 2017 13:20:26 +0000 (15:20 +0200)
The test suite contains multiple instances of determining the target's
endianness with GDB's "show endian" command.  This patch replaces these by
an invocation of a new convenience proc 'get_endianness'.

gdb/testsuite/ChangeLog:

* lib/gdb.exp (get_endianness): New proc.
* gdb.arch/aarch64-fp.exp: Use it.
* gdb.arch/altivec-regs.exp: Likewise.
* gdb.arch/e500-regs.exp: Likewise.
* gdb.arch/vsx-regs.exp: Likewise.
* gdb.base/dump.exp: Likewise.
* gdb.base/funcargs.exp: Likewise.
* gdb.base/gnu_vector.exp: Likewise.
* gdb.dwarf2/formdata16.exp: Likewise.
* gdb.dwarf2/implptrpiece.exp: Likewise.
* gdb.dwarf2/nonvar-access.exp: Likewise.
* gdb.python/py-inferior.exp: Likewise.
* gdb.trace/unavailable-dwarf-piece.exp: Likewise.

14 files changed:
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.arch/aarch64-fp.exp
gdb/testsuite/gdb.arch/altivec-regs.exp
gdb/testsuite/gdb.arch/e500-regs.exp
gdb/testsuite/gdb.arch/vsx-regs.exp
gdb/testsuite/gdb.base/dump.exp
gdb/testsuite/gdb.base/funcargs.exp
gdb/testsuite/gdb.base/gnu_vector.exp
gdb/testsuite/gdb.dwarf2/formdata16.exp
gdb/testsuite/gdb.dwarf2/implptrpiece.exp
gdb/testsuite/gdb.dwarf2/nonvar-access.exp
gdb/testsuite/gdb.python/py-inferior.exp
gdb/testsuite/gdb.trace/unavailable-dwarf-piece.exp
gdb/testsuite/lib/gdb.exp

index 882ec3ae73c824805af1f448232938d95e3f0665..878d08de607073df8417567605db4abc22c28541 100644 (file)
@@ -1,3 +1,19 @@
+2017-06-13  Andreas Arnez  <arnez@linux.vnet.ibm.com>
+
+       * lib/gdb.exp (get_endianness): New proc.
+       * gdb.arch/aarch64-fp.exp: Use it.
+       * gdb.arch/altivec-regs.exp: Likewise.
+       * gdb.arch/e500-regs.exp: Likewise.
+       * gdb.arch/vsx-regs.exp: Likewise.
+       * gdb.base/dump.exp: Likewise.
+       * gdb.base/funcargs.exp: Likewise.
+       * gdb.base/gnu_vector.exp: Likewise.
+       * gdb.dwarf2/formdata16.exp: Likewise.
+       * gdb.dwarf2/implptrpiece.exp: Likewise.
+       * gdb.dwarf2/nonvar-access.exp: Likewise.
+       * gdb.python/py-inferior.exp: Likewise.
+       * gdb.trace/unavailable-dwarf-piece.exp: Likewise.
+
 2017-06-13  Andreas Arnez  <arnez@linux.vnet.ibm.com>
 
        PR gdb/21226
index 2faf121a33d429185726199058dfa2bd8f26bf40..483c8f1816fe4e3934a2f86ae6705238c6abad39 100644 (file)
@@ -34,14 +34,7 @@ if ![runto_main] {
     return -1
 }
 
-set endianness "little"
-set test "show endian"
-gdb_test_multiple $test $test {
-    -re "(.* )(big|little)( endian.*)$gdb_prompt $" {
-        set endianness $expect_out(2,string)
-        pass "endianness"
-    }
-}
+set endianness [get_endianness]
 
 gdb_test "break ${srcfile}:[gdb_get_line_number "return"]" \
     "Breakpoint $decimal at 0x\[0-9a-fA-F\]+: file .*${srcfile}.*\\\." \
index b3b5aed5db51ad761e924e65829bc5c4ef367532..165f61a194a5ded4435a795176b3b8035f1b1b58 100644 (file)
@@ -77,17 +77,7 @@ gdb_test "set \$vrsave = 1" "" ""
 
 gdb_test "next" "" ""
 
-set endianness ""
-set msg "detect endianness"
-gdb_test_multiple "show endian" "$msg" {
-    -re "(The target endianness is set automatically .currently )(big|little)( endian.*)$gdb_prompt $" {
-        pass "$msg"
-        set endianness $expect_out(2,string)
-    }
-    -re ".*$gdb_prompt $" {
-        fail "$msg"
-    }
-}
+set endianness [get_endianness]
 
 # And then read the AltiVec registers back, to see that
 # a) the register write above worked, and
index 8bf405a9afe969ed976f40261262e2c6af868e9e..bab6e7ddbd2df36188cbe1600d2c9cfd776ff6bc 100644 (file)
@@ -59,17 +59,7 @@ for {set i 0} {$i < 32} {incr i 1} {
 
 #gdb_test "next" "" ""
 
-send_gdb "show endian\n"
-gdb_expect {
-    -re "(The target endianness is set automatically .currently )(big|little)( endian.*)$gdb_prompt $" {
-        pass "endianness"
-       set endianness $expect_out(2,string)
-    }
-    -re ".*$gdb_prompt $" {
-       fail "couldn't get endianness"
-    }
-    timeout            { fail "(timeout) endianness" }
-}
+set endianness [get_endianness]
 
 # And then read the E500 registers back, to see that
 # a) the register write above worked, and
index 15e7196367aaaa3e03c374ecc3563976d1add777..e6b8d726366bd92cae84098bc8195feec6a7ef4b 100644 (file)
@@ -56,17 +56,7 @@ if ![runto_main] then {
     gdb_suppress_tests
 }
 
-set endianness ""
-set msg "detect endianness"
-gdb_test_multiple "show endian" "$msg" {
-    -re "(The target endianness is set automatically .currently )(big|little)( endian.*)$gdb_prompt $" {
-        pass "$msg"
-        set endianness $expect_out(2,string)
-    }
-    -re ".*$gdb_prompt $" {
-        fail "$msg"
-    }
-}
+set endianness [get_endianness]
 
 # Data sets used throughout the test
 
index e67e1aa91e3e09b42f33a091e2de54e40d797d4d..a9625d754b4894e12dab1af28cc6b75d81326686 100644 (file)
@@ -101,12 +101,7 @@ if { ! [ runto checkpoint1 ] } then {
 
 # Get the endianness for the later use with endianless formats.
 
-gdb_test_multiple "show endian" "show endian" {
-    -re ".* (big|little) endian.*$gdb_prompt $" { 
-       set endian $expect_out(1,string) 
-       pass "endianness: $endian"
-    }
-}
+set endian [get_endianness]
 
 # Now generate some dump files.
 
index 5852f12b9369e5cbe2cfac780ce7edb3b1515d81..c48b88203e9554c104a89c1982b39a39034907a1 100644 (file)
@@ -1139,15 +1139,9 @@ gdb_test_multiple "print sizeof (int)" "getting sizeof int" {
     }
 }
 
-gdb_test_multiple "show endian" "getting target endian" {
-    -re ".*little endian.*$gdb_prompt $" {
-       set target_bigendian_p 0
-       # pass silently
-    }
-    -re ".*big endian.*$gdb_prompt $" {
-       set target_bigendian_p 1
-       # pass silently
-    }
+switch [get_endianness] {
+    little { set target_bigendian_p 0 }
+    big { set target_bigendian_p 1 }
 }
 
 # Perform tests
index 1e0e46c2df20571bedb511425dc5b108381ebb65..44b1405107a1d57a4a5255a01d1541b913ca387a 100644 (file)
@@ -48,12 +48,7 @@ if { ![runto main] } {
 }
 
 # Get endianess for the scalar->vector casts
-gdb_test_multiple "show endian" "show endian" {
-    -re ".* (big|little) endian.*$gdb_prompt $" { 
-       set endian $expect_out(1,string) 
-       pass "endianness: $endian"
-    }
-}
+set endian [get_endianness]
 
 # Test printing of character vector types
 gdb_test "print c4" "\\\$$decimal = \\{1, 2, 3, 4\\}"
index a3ccbd39b50066db75c4da08ba6fb26cd38481f1..752ccb4a7d52af7e9c2a61ffe5279cd541f0c3b3 100644 (file)
@@ -28,14 +28,7 @@ if [prepare_for_testing "failed to prepare for endianness test" ${testfile} ${sr
     return -1
 }
 
-set endianness "little"
-set test "show endian"
-gdb_test_multiple $test $test {
-    -re "(.* )(big|little)( endian.*)\r\n$gdb_prompt $" {
-       set endianness $expect_out(2,string)
-       pass "endianness"
-    }
-}
+set endianness [get_endianness]
 
 set high "0x123456789abcdef0"
 set low "0x0fedcba987654321"
index ef483c0f98001ea3d373bcd630f761f4f0f6d41c..ac3ee7846599e301d920112aadfc99747aae3332 100644 (file)
@@ -119,14 +119,8 @@ if ![runto_main] {
     return -1
 }
 
-# Determine endianness.
-set endian "little"
-gdb_test_multiple "show endian" "determine endianness" {
-    -re ".* (big|little) endian.*$gdb_prompt $" {
-       set endian $expect_out(1,string)
-       pass "endianness: $endian"
-    }
-}
+# Determine byte order.
+set endian [get_endianness]
 
 # Access the second byte of s through an implicit pointer to the third
 # byte of s, using a negative offset.  Compare that to the second byte of
index 54060291b39ba73e2240760b097cfb56eef38f73..b7d952a5f28207701fb425c6cc33e67091098167 100644 (file)
@@ -205,14 +205,8 @@ if ![runto_main] {
     return -1
 }
 
-# Determine endianness.
-set endian "little"
-gdb_test_multiple "show endian" "determine endianness" {
-    -re ".* (big|little) endian.*$gdb_prompt $" {
-       set endian $expect_out(1,string)
-       pass "endianness: $endian"
-    }
-}
+# Determine byte order.
+set endian [get_endianness]
 
 # Byte-aligned objects with simple location descriptions.
 switch $endian { big {set val 0x345678} little {set val 0x785634} }
index adb5df3503258eaa36738782584ef47e97225b56..c2ea2c2a4bc9e4dacc791d5d993e3a00f3ee4e91 100644 (file)
@@ -30,15 +30,9 @@ clean_restart ${testfile}
 # Skip all tests if Python scripting is not enabled.
 if { [skip_python_tests] } { continue }
 
-gdb_test_multiple "show endian" "getting target endian" {
-    -re ".*little endian.*$gdb_prompt $" {
-        set python_pack_char "<"
-       # pass silently
-    }
-    -re ".*big endian.*$gdb_prompt $" {
-        set python_pack_char ">"
-       # pass silently
-    }
+switch [get_endianness] {
+    little { set python_pack_char "<" }
+    big { set python_pack_char ">" }
 }
 
 # The following tests require execution.
index 3b14bea0703e724d91a409f2b077f2c19db6d723..d8a75d4809b716f40d424711685159b1d15e676d 100644 (file)
@@ -325,13 +325,7 @@ with_test_prefix "tracing bar" {
     gdb_test "continue" "Continuing\\.\[ \r\n\]+Breakpoint.*"
     gdb_test_no_output "tstop"
 
-    set endian ""
-    gdb_test_multiple "show endian" "show endian" {
-       -re ".* (big|little) endian.*$gdb_prompt $" {
-           set endian $expect_out(1,string)
-           pass "endianness: $endian"
-       }
-    }
+    set endian [get_endianness]
 
     gdb_test "tfind 0" "Found trace frame 0, tracepoint .*"
     if { $endian == "little" } {
index 6633d2429650f75f19553ef6ddb9082c58c18269..a74f97465ec07282df250910476c107fc4567f34 100644 (file)
@@ -5671,6 +5671,19 @@ proc set_remotetimeout { timeout } {
     }
 }
 
+# Get the target's current endianness and return it.
+proc get_endianness { } {
+    global gdb_prompt
+
+    gdb_test_multiple "show endian" "determine endianness" {
+       -re ".* (little|big) endian.*\r\n$gdb_prompt $" {
+           # Pass silently.
+           return $expect_out(1,string)
+       }
+    }
+    return "little"
+}
+
 # ROOT and FULL are file names.  Returns the relative path from ROOT
 # to FULL.  Note that FULL must be in a subdirectory of ROOT.
 # For example, given ROOT = /usr/bin and FULL = /usr/bin/ls, this