public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] sim: testsuite: drop sim_compile cover function
@ 2021-11-11  8:28 Mike Frysinger
  0 siblings, 0 replies; only message in thread
From: Mike Frysinger @ 2021-11-11  8:28 UTC (permalink / raw)
  To: gdb-patches

Most code isn't using this, and the only call site (in one cris file)
can use target_compile directly.  So switch it over to simplify.
---
 sim/testsuite/cris/c/c.exp     |  2 +-
 sim/testsuite/lib/sim-defs.exp | 13 -------------
 2 files changed, 1 insertion(+), 14 deletions(-)

diff --git a/sim/testsuite/cris/c/c.exp b/sim/testsuite/cris/c/c.exp
index 08ea18812ec5..26084682dc95 100644
--- a/sim/testsuite/cris/c/c.exp
+++ b/sim/testsuite/cris/c/c.exp
@@ -189,7 +189,7 @@ foreach src [lsort [glob -nocomplain $srcdir/$subdir/*.c]] {
     verbose -log "Compiling $src with $opts(cc)"
 
     set dest "$opts(dest)"
-    if { [sim_compile $src $dest "executable" "$opts(cc)" ] != "" } {
+    if { [target_compile $src $dest "executable" "$opts(cc)" ] != "" } {
 	unresolved $testname
 	continue
     }
diff --git a/sim/testsuite/lib/sim-defs.exp b/sim/testsuite/lib/sim-defs.exp
index 6e8feaf6bf89..7bc4c66549ee 100644
--- a/sim/testsuite/lib/sim-defs.exp
+++ b/sim/testsuite/lib/sim-defs.exp
@@ -35,19 +35,6 @@ proc sim_version {} {
     clone_output "$sim_path $version\n"
 }
 
-# Cover function to target_compile.
-# Copied from gdb_compile.
-
-proc sim_compile { source dest type options } {
-    set result [target_compile $source $dest $type $options]
-    regsub "\[\r\n\]*$" "$result" "" result
-    regsub "^\[\r\n\]*" "$result" "" result
-    if { $result != "" } {
-	clone_output "sim compile output: $result"
-    }
-    return $result
-}
-
 # Find the path to the simulator for executing.
 proc sim_tool_path {} {
     global sim_path
-- 
2.33.0


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2021-11-11  8:28 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-11  8:28 [PATCH] sim: testsuite: drop sim_compile cover function Mike Frysinger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).