Don't skip catch-syscall.exp on hppa*-hp-hpux* target

Message ID 1425033548-1003-1-git-send-email-qiyaoltc@gmail.com
State New, archived
Headers

Commit Message

Yao Qi Feb. 27, 2015, 10:39 a.m. UTC
  From: Yao Qi <yao.qi@linaro.org>

As far as I know, "catch syscall" is supported on hppa*-hp-hpux*, but
the test catch-syscall.exp is skipped on this target by mistake.  This
patch is to fix it.  However, I don't have a hpux machine to test.

gdb/testsuite:

2015-02-27  Yao Qi  <yao.qi@linaro.org>

	* gdb.base/catch-syscall.exp: Don't skip it on hppa*-hp-hpux*
	target.
---
 gdb/testsuite/gdb.base/catch-syscall.exp | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)
  

Comments

Pedro Alves Feb. 27, 2015, 12:36 p.m. UTC | #1
On 02/27/2015 10:39 AM, Yao Qi wrote:
> From: Yao Qi <yao.qi@linaro.org>
> 
> As far as I know, "catch syscall" is supported on hppa*-hp-hpux*, but
> the test catch-syscall.exp is skipped on this target by mistake.  This
> patch is to fix it.  However, I don't have a hpux machine to test.

LGTM.

Thanks,
Pedro Alves
  
Yao Qi Feb. 27, 2015, 1:46 p.m. UTC | #2
On 27/02/15 12:36, Pedro Alves wrote:
> On 02/27/2015 10:39 AM, Yao Qi wrote:
>> From: Yao Qi <yao.qi@linaro.org>
>>
>> As far as I know, "catch syscall" is supported on hppa*-hp-hpux*, but
>> the test catch-syscall.exp is skipped on this target by mistake.  This
>> patch is to fix it.  However, I don't have a hpux machine to test.
>
> LGTM.
>

Thanks, patch is pushed in.
  

Patch

diff --git a/gdb/testsuite/gdb.base/catch-syscall.exp b/gdb/testsuite/gdb.base/catch-syscall.exp
index 33f5354..8f9dec3 100644
--- a/gdb/testsuite/gdb.base/catch-syscall.exp
+++ b/gdb/testsuite/gdb.base/catch-syscall.exp
@@ -23,19 +23,13 @@  if { [is_remote target] || ![isnative] } then {
     continue
 }
 
-# Until "catch syscall" is implemented on other targets...
-if { ![istarget "hppa*-hp-hpux*"] && ![istarget "*-linux*"] } {
-    continue
-}
-
 # This shall be updated whenever 'catch syscall' is implemented
 # on some architecture.
-#if { ![istarget "i\[34567\]86-*-linux*"]
 if { ![istarget "x86_64-*-linux*"] && ![istarget "i\[34567\]86-*-linux*"]
      && ![istarget "powerpc-*-linux*"] && ![istarget "powerpc64-*-linux*"]
      && ![istarget "sparc-*-linux*"] && ![istarget "sparc64-*-linux*"]
      && ![istarget "mips*-linux*"] && ![istarget "arm*-linux*"]
-     && ![istarget "s390*-linux*"] } {
+     && ![istarget "s390*-linux*"] && ![istarget "hppa*-hp-hpux*"] } {
      continue
 }