[PATCH OBV] Don't skip hw breakpoint/watchpoint tests for aarch64 target

[PATCH OBV] Don't skip hw breakpoint/watchpoint tests for aarch64 target

[Yao Qi]

This patch is to let skip_hw_breakpoint_tests and skip_hw_watchpoint_tests
return 0 for aarch64 target, since aarch64 has HW watchpoint and
breakpoint registers.

With this patch applied, about 1560 watchpoint/breakpoint related tests
become enabled on aarch64-linux native testing.

It is obvious, and I'll push it in.

gdb/testsuite:

2015-06-22  Yao Qi  <yao.qi@linaro.org>

	* lib/gdb.exp (skip_hw_breakpoint_tests): Return 0 for target
	aarch64*-*-*.
	(skip_hw_watchpoint_tests): Likewise.
---
 gdb/testsuite/ChangeLog   | 6 ++++++
 gdb/testsuite/lib/gdb.exp | 4 +++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 5a9a7eb..3a48bcd 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2015-06-22  Yao Qi  <yao.qi@linaro.org>
+
+	* lib/gdb.exp (skip_hw_breakpoint_tests): Return 0 for target
+	aarch64*-*-*.
+	(skip_hw_watchpoint_tests): Likewise.
+
 2015-06-18  Patrick Palka  <patrick@parcs.ath.cx>
 
 	* gdb.base/gdbinit-history.exp: Test the interaction between
diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index a3d570a..d169f3d 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -2591,7 +2591,8 @@ proc skip_hw_breakpoint_tests {} {
     if { [istarget "i?86-*-*"] 
 	 || [istarget "x86_64-*-*"]
 	 || [istarget "ia64-*-*"] 
-	 || [istarget "arm*-*-*"]} {
+	 || [istarget "arm*-*-*"]
+	 || [istarget "aarch64*-*-*"]} {
 	return 0
     }
 
@@ -2611,6 +2612,7 @@ proc skip_hw_watchpoint_tests {} {
 	 || [istarget "x86_64-*-*"]
 	 || [istarget "ia64-*-*"] 
 	 || [istarget "arm*-*-*"]
+	 || [istarget "aarch*-*-*"]
 	 || [istarget "powerpc*-*-linux*"]
 	 || [istarget "s390*-*-*"] } {
 	return 0
-- 
1.9.1

Re: [PATCH OBV] Don't skip hw breakpoint/watchpoint tests for aarch64 target

[Yao Qi]

On 22/06/15 13:40, Yao Qi wrote:
> @@ -2611,6 +2612,7 @@ proc skip_hw_watchpoint_tests {} {
>   	 || [istarget "x86_64-*-*"]
>   	 || [istarget "ia64-*-*"]
>   	 || [istarget "arm*-*-*"]
> +	 || [istarget "aarch*-*-*"]

Sorry, it should be "aarch64*-*-*".  Patch below is pushed in.

-- 
Yao (齐尧)

gdb/testsuite:

2015-06-22  Yao Qi  <yao.qi@linaro.org>

	* lib/gdb.exp (skip_hw_breakpoint_tests): Return 0 for target
	aarch64*-*-*.
	(skip_hw_watchpoint_tests): Likewise.

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 5a9a7eb..3a48bcd 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2015-06-22  Yao Qi  <yao.qi@linaro.org>
+
+	* lib/gdb.exp (skip_hw_breakpoint_tests): Return 0 for target
+	aarch64*-*-*.
+	(skip_hw_watchpoint_tests): Likewise.
+
  2015-06-18  Patrick Palka  <patrick@parcs.ath.cx>

  	* gdb.base/gdbinit-history.exp: Test the interaction between
diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index b5928c3..9edf2dc 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -2556,7 +2556,8 @@ proc skip_hw_breakpoint_tests {} {
      if { [istarget "i?86-*-*"]
  	 || [istarget "x86_64-*-*"]
  	 || [istarget "ia64-*-*"]
-	 || [istarget "arm*-*-*"]} {
+	 || [istarget "arm*-*-*"]
+	 || [istarget "aarch64*-*-*"]} {
  	return 0
      }

@@ -2576,6 +2577,7 @@ proc skip_hw_watchpoint_tests {} {
  	 || [istarget "x86_64-*-*"]
  	 || [istarget "ia64-*-*"]
  	 || [istarget "arm*-*-*"]
+	 || [istarget "aarch64*-*-*"]
  	 || [istarget "powerpc*-*-linux*"]
  	 || [istarget "s390*-*-*"] } {
  	return 0