public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] Fix compilation of dyn_arrayidx ada testcase
@ 2015-01-25  8:02 Doug Evans
  2015-01-29 15:58 ` Joel Brobecker
  0 siblings, 1 reply; 2+ messages in thread
From: Doug Evans @ 2015-01-25  8:02 UTC (permalink / raw)
  To: brobecker, gdb-patches

Hi.

I was looking into this testcase as boilerplate for another test
I want to write for a cleanup patch, and noticed it doesn't compile.

gcc -c -m64 -g -lm -I- -gnatA /hack/build/vptr-cleanup-1/vptr-cleanup-1/gdb/testsuite/gdb.ada/dyn_arrayidx/foo.adb
foo.adb:23:13: iterator is an Ada 2012 feature

2015-01-24  Doug Evans  <xdje42@gmail.com>

	* gdb.ada/dyn_arrayidx.exp: Add additional_flags=-gnat12.

diff --git a/gdb/testsuite/gdb.ada/dyn_arrayidx.exp b/gdb/testsuite/gdb.ada/dyn_arrayidx.exp
index 25351db..06702b8 100644
--- a/gdb/testsuite/gdb.ada/dyn_arrayidx.exp
+++ b/gdb/testsuite/gdb.ada/dyn_arrayidx.exp
@@ -19,7 +19,7 @@ if { [skip_ada_tests] } { return -1 }
 
 standard_ada_testfile foo
 
-if {[gdb_compile_ada "${srcfile}" "${binfile}" executable {debug}] != ""} {
+if {[gdb_compile_ada "${srcfile}" "${binfile}" executable {debug additional_flags=-gnat12}] != ""} {
     return -1
 }
 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] Fix compilation of dyn_arrayidx ada testcase
  2015-01-25  8:02 [PATCH] Fix compilation of dyn_arrayidx ada testcase Doug Evans
@ 2015-01-29 15:58 ` Joel Brobecker
  0 siblings, 0 replies; 2+ messages in thread
From: Joel Brobecker @ 2015-01-29 15:58 UTC (permalink / raw)
  To: Doug Evans; +Cc: gdb-patches

> 2015-01-24  Doug Evans  <xdje42@gmail.com>
> 
> 	* gdb.ada/dyn_arrayidx.exp: Add additional_flags=-gnat12.

Thanks for the fix, Doug. I didn't see this, because we changed our
compiler to build with -gnat12 by default. This is an Ada Front End
change, so I would assume that this made it to the FSF GCC by now;
just don't know which version.

Looking at the testcase, it would have been easy to adjust it to avoid
that Ada 2012 feature. But since your fix is really easy, might as well
keep it that way.

OK to push if you haven't already.

> diff --git a/gdb/testsuite/gdb.ada/dyn_arrayidx.exp b/gdb/testsuite/gdb.ada/dyn_arrayidx.exp
> index 25351db..06702b8 100644
> --- a/gdb/testsuite/gdb.ada/dyn_arrayidx.exp
> +++ b/gdb/testsuite/gdb.ada/dyn_arrayidx.exp
> @@ -19,7 +19,7 @@ if { [skip_ada_tests] } { return -1 }
>  
>  standard_ada_testfile foo
>  
> -if {[gdb_compile_ada "${srcfile}" "${binfile}" executable {debug}] != ""} {
> +if {[gdb_compile_ada "${srcfile}" "${binfile}" executable {debug additional_flags=-gnat12}] != ""} {
>      return -1
>  }
>  

-- 
Joel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-01-29  7:42 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-25  8:02 [PATCH] Fix compilation of dyn_arrayidx ada testcase Doug Evans
2015-01-29 15:58 ` Joel Brobecker

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).