This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: [gnat] reuse of ASTs already constructed
- From: oliver dot kellogg at t-online dot de (Oliver Kellogg)
- To: report at gnat dot com
- Cc: gcc at gcc dot gnu dot org
- Date: Mon, 15 Jun 2009 22:45:31 +0200
- Subject: Re: [gnat] reuse of ASTs already constructed
- References: <1239557374.7565.37.camel@tidbit.site> <1240083309.4554.45.camel@tidbit.site> <1240175954.4554.50.camel@tidbit.site> <1241385654.4763.38.camel@tidbit.site> <49FE09D9.4090702@adacore.com> <1241386066.4763.42.camel@tidbit.site> <49FE415A.5050601@adacore.com> <1241415395.21122.14.camel@tidbit.site> <1241899436.5909.14.camel@tidbit.site> <4A05E2CF.6010708@adacore.com>
- Reply-to: okellogg at users dot sourceforge dot net
A further problem that I'm seeing is that
Lib.Writ.Write_Unit_Information assumes the current unit to depend on
all entries in Lib.Units. However, when letting Lib.Units accumulate
units across several compilations this dependency does not
necessarily hold true. The generated ALI files for compilation 2
to N contain more entries than needed.
--Oliver
On Sat, 2009-05-09 at 16:08 -0400, Robert Dewar wrote:
> Oliver Kellogg wrote:
> > On 2009-05-04, at 07:36 +0200, Oliver Kellogg wrote:
> >> Robert Dewar wrote:
> >>>>>>> How about not doing the name expansion in-place but rather
> >>>>>>> storing the expanded name in an extra node field?
> >>> You could have a separate vector for expanded names I suppose ...
> >> So be it. I will change the code to not overwrite the Name field
> >> with the expanded name but rather store the expanded name in a
> >> data structure separate from the Node.
> >>
> >
> > Luckily, this change was limited to the Exp_Dbug package body.
> > FYI I am appending the diff.
> > The new Q_E_Name array holds the Name_Id for the expanded name
> > of a given entity. The implementation as a fixed array is
> > preliminary and I would appreciate suggestions on a better
> > data structure to use.
>
> use a table .. we never use fixed length arrays in the compiler