[Ada] Fix -fdump-ada-spec issue with array field

2019-11-13 Thread Eric Botcazou
This is a regression present on mainline and 9 branch: if the component type of array type is a structure declared in another file, then the binding would contain a local declaration for this structure type. Tested on x86_64-suse-linux, applied on the mainline and 9 branch. 2019-11-13 Eric Bo

[c-family] Fix -fdump-ada-spec regressions in C++

2018-11-27 Thread Eric Botcazou
This fixes a few regressions introduced by the switch to Ada 2012 for the output of -fdump-ada-spec that are present when the input is in C++. Tested on x86_64-suse-linux, applied on the mainline. 2018-11-27 Eric Botcazou * c-ada-spec.c: Include stringpool.h. (has_static_fie

[c-family] Fix -fdump-ada-spec ordering issue in C++

2015-11-30 Thread Eric Botcazou
This fixes an ordering issue in the Ada code generated by the -fdump-ada-spec option with the C++ compiler on structures/unions with nested anonymous arrays of structures/unions. Given that this only affects the Ada code generated by -fdump-ada-spec and has no effect whatsoever on the C and C++

Re: [c-family] Fix -fdump-ada-spec ICEs

2015-03-06 Thread Eric Botcazou
> Following this commit (r221088) testing dump-ada-spec-3.C with > > make -k check-g++ RUNTESTFLAGS="dg.exp=other/dump-ada-spec-3.C > > generates a lot of *.ads files in the gcc/testsuite/g++ directory > which are not cleaned up after completion. Sorry about that, I thought cleanup-ada-spec woul

Re: [c-family] Fix -fdump-ada-spec ICEs

2015-03-05 Thread Dominique Dhumieres
Hi Eric, Following this commit (r221088) testing dump-ada-spec-3.C with make -k check-g++ RUNTESTFLAGS="dg.exp=other/dump-ada-spec-3.C generates a lot of *.ads files in the gcc/testsuite/g++ directory which are not cleaned up after completion. Any idea about how to do the cleaning? TIA Domini

[c-family] Fix -fdump-ada-spec ICEs

2015-03-01 Thread Eric Botcazou
They were introduced by the latest tree structure re-shuffling. Tested on x86_64-suse-linux, applied on the mainline as obvious. 2015-03-01 Eric Botcazou * c-ada-spec.c (is_tagged_type): Add guard for DECL_VINDEX. (dump_template_types): Adjust DECL_TEMPLATE_INSTANTIATIONS and

Re: [c-family] Fix -fdump-ada-spec buglet in C++

2012-10-15 Thread Joseph S. Myers
On Mon, 15 Oct 2012, Eric Botcazou wrote: > Hi, > > since the sizetype change, we generate invalid Ada for flexible array members > with -fdump-ada-spec in C++. The attached patch fixes this issue and also > partially revamps the code to polish some rough edges. > > Tested on x86_64-suse-linu

[c-family] Fix -fdump-ada-spec buglet in C++

2012-10-15 Thread Eric Botcazou
Hi, since the sizetype change, we generate invalid Ada for flexible array members with -fdump-ada-spec in C++. The attached patch fixes this issue and also partially revamps the code to polish some rough edges. Tested on x86_64-suse-linux, OK for mainline? 2012-10-15 Eric Botcazou c-fami

Re: Fix -fdump-ada-spec

2012-10-04 Thread Richard Guenther
On Thu, Oct 4, 2012 at 10:26 AM, Arnaud Charlet wrote: > After changes by Sharad (Add option for dumping to stderr (issue6190057)), > -fdump-ada-spec is broken, and is now a no-op. > > Admittedly, this is because -fdump-ada-spec is handled differently from > other -fdump-* switches, so this patch

Fix -fdump-ada-spec

2012-10-04 Thread Arnaud Charlet
After changes by Sharad (Add option for dumping to stderr (issue6190057)), -fdump-ada-spec is broken, and is now a no-op. Admittedly, this is because -fdump-ada-spec is handled differently from other -fdump-* switches, so this patch fixes support for -fdump-ada-spec by using an approach similar to

Re: PATCH: fix -fdump-ada-spec after PR debug/51410

2012-01-06 Thread Eric Botcazou
> Fixed by now taking into account ext_block in collect_all_refs and > for_each_global_decl (and moving the declaration of ext_block up in the > file). > > Tested on i686-pc-linux-gnu, OK for trunk? Since it only affects -fdump-ada-spec, I took the liberty to backport it to the 4.6 branch, as Jak

Re: PATCH: fix -fdump-ada-spec after PR debug/51410

2012-01-06 Thread Joseph S. Myers
On Fri, 6 Jan 2012, Arnaud Charlet wrote: > 2011-01-06 Arnaud Charlet > > * c-decl.c (ext_block): Moved up. > (collect_all_refs, for_each_global_decl): Take ext_block into account. OK. -- Joseph S. Myers jos...@codesourcery.com

PATCH: fix -fdump-ada-spec after PR debug/51410

2012-01-06 Thread Arnaud Charlet
The fix for PR debug/51410 (http://gcc.gnu.org/PR51410) available at: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg00375.html has introduced a regression in the handling of -fdump-ada-spec which was taking the previous behavior wrt ext_block into account to avoid generating twice the same declara