gdb/testsuite: remove get_compiler_info calls from gdb.exp and dwarf.exp
authorAndrew Burgess <aburgess@redhat.com>
Wed, 8 Jun 2022 13:25:00 +0000 (14:25 +0100)
committerAndrew Burgess <aburgess@redhat.com>
Thu, 9 Jun 2022 13:40:48 +0000 (14:40 +0100)
We don't need to call get_compiler_info before calling
test_compiler_info; test_compiler_info includes a call to
get_compiler_info.

This commit cleans up lib/gdb.exp and lib/dwarf.exp a little by
removing some unneeded calls to get_compiler_info.  We could do the
same cleanup throughout the testsuite, but I'm leaving that for
another day.

There should be no change in the test results after this commit.

gdb/testsuite/lib/dwarf.exp
gdb/testsuite/lib/gdb.exp

index b05fdd58f6616a7fd0f83ea3408207cb34ab432c..3d0ea83aee79aba143dd92a9af3436b79dd0864a 100644 (file)
@@ -110,14 +110,6 @@ proc build_executable_and_dwo_files { testname executable options args } {
        set binfile $executable
     }
 
-    set info_options ""
-    if { [lsearch -exact $options "c++"] >= 0 } {
-       set info_options "c++"
-    }
-    if [get_compiler_info ${info_options}] {
-        return -1
-    }
-
     set func gdb_compile
     if {[lsearch -regexp $options \
             {^(pthreads|shlib|shlib_pthreads|openmp)$}] != -1} {
index 7b11dab870e1eeed2c597fe4723f0cad669af8ec..c9a30d88b2f734f9e20663c4cb0725c2fa394eaf 100644 (file)
@@ -3238,10 +3238,6 @@ gdb_caching_proc skip_altivec_tests {
     }
 
     # Make sure we have a compiler that understands altivec.
-    if [get_compiler_info] {
-       warning "Could not get compiler info"
-       return 1
-    }
     if [test_compiler_info gcc*] {
         set compile_flags "additional_flags=-maltivec"
     } elseif [test_compiler_info xlc*] {
@@ -3357,10 +3353,6 @@ gdb_caching_proc skip_vsx_tests {
     }
 
     # Make sure we have a compiler that understands altivec.
-    if [get_compiler_info] {
-       warning "Could not get compiler info"
-       return 1
-    }
     if [test_compiler_info gcc*] {
         set compile_flags "additional_flags=-mvsx"
     } elseif [test_compiler_info xlc*] {
@@ -4781,11 +4773,8 @@ proc gdb_compile_shlib_1 {sources dest options} {
     } elseif { [lsearch -exact $options "f90"] >= 0 } {
        set info_options "f90"
     }
-    if [get_compiler_info ${info_options}] {
-       return -1
-    }
 
-    switch -glob [test_compiler_info] {
+    switch -glob [test_compiler_info "" ${info_options}] {
         "xlc-*" {
             lappend obj_options "additional_flags=-qpic"
         }
@@ -6964,16 +6953,6 @@ proc build_executable_from_specs {testname executable options args} {
 
     set binfile [standard_output_file $executable]
 
-    set info_options ""
-    if { [lsearch -exact $options "c++"] >= 0 } {
-       set info_options "c++"
-    } elseif { [lsearch -exact $options "f90"] >= 0 } {
-       set info_options "f90"
-    }
-    if [get_compiler_info ${info_options}] {
-        return -1
-    }
-
     set func gdb_compile
     set func_index [lsearch -regexp $options {^(pthreads|shlib|shlib_pthreads|openmp)$}]
     if {$func_index != -1} {