Skip to content

Commit

Permalink
Check -fvar-tracking via ada_simple_compile
Browse files Browse the repository at this point in the history
A couple of Ada tests check whether the C compiler supports
-fvar-tracking.  However, this doesn't really work when using
gnat-llvm, because that will invoke clang under the hood.  This patch
arranges to check gnatmake instead, which is more robust even when
toolchains are mix-and-matched.
  • Loading branch information
tromey committed Mar 4, 2025
1 parent 6a509dd commit e0386de
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 2 deletions.
2 changes: 1 addition & 1 deletion gdb/testsuite/gdb.ada/array_return.exp
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ standard_ada_testfile p

set opts {}
lappend opts debug
if { [have_fvar_tracking] } {
if { [ada_fvar_tracking] } {
lappend opts "additional_flags=-fvar-tracking"
}

Expand Down
2 changes: 1 addition & 1 deletion gdb/testsuite/gdb.ada/finish-large.exp
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ standard_ada_testfile p

set flags {debug}

if {[have_fvar_tracking]} {
if {[ada_fvar_tracking]} {
lappend flags "additional_flags=-fvar-tracking"
}

Expand Down
5 changes: 5 additions & 0 deletions gdb/testsuite/lib/ada.exp
Original file line number Diff line number Diff line change
Expand Up @@ -263,3 +263,8 @@ proc ada_simple_compile {name options} {
set output [gdb_compile_ada_1 $src $dest executable $options]
return [expr {[gdb_compile_test_nofail $output] == 1}]
}

# Return 1 if GNAT supports -fvar-tracking.
gdb_caching_proc ada_fvar_tracking {} {
return [ada_simple_compile fvar_tracking additional_flags=-fvar-tracking]
}

0 comments on commit e0386de

Please sign in to comment.