gdb/testsuite: Rename skip_hipcc_tests to allow_hipcc_tests

Rename skip_hipcc_tests to allow_hipcc_tests so it can be used as a
"require" predicate in tests.

Use require in gdb.rocm/simple.exp.

Approved-By: Simon Marchi <simon.marchi@efficios.com>
This commit is contained in:
Lancelot SIX
2023-02-06 18:05:31 +00:00
parent 08d8af48e4
commit 09ad7eb8cc
2 changed files with 4 additions and 7 deletions

View File

@@ -20,10 +20,7 @@ load_lib rocm.exp
standard_testfile .cpp
if [skip_hipcc_tests] {
verbose "skipping hip test: ${testfile}"
return
}
require allow_hipcc_tests
if {[build_executable "failed to prepare" $testfile $srcfile {debug hip}]} {
return

View File

@@ -15,14 +15,14 @@
#
# Support library for testing ROCm (AMD GPU) GDB features.
proc skip_hipcc_tests { } {
proc allow_hipcc_tests { } {
# Only the native target supports ROCm debugging. E.g., when
# testing against GDBserver, there's no point in running the ROCm
# tests.
if {[target_info gdb_protocol] != ""} {
return 1
return 0
}
return 0
return 1
}
# The lock file used to ensure that only one GDB has access to the GPU