This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, TREE_LIST removal] remove calls to build_constructor_from_list from coverage.c
On Thu, May 13, 2010 at 07:46:24PM -0700, Nathan Froyd wrote:
> > > On Thu, May 13, 2010 at 09:13:29AM -0600, Tom Tromey wrote:
> > >> So I think another approach would be to change START_RECORD_CONSTRUCTOR
> > >> to simply initialize the VEC, then modify PUSH_FIELD_VALUE, PUSH_SUPER_VALUE,
> > >> and FINISH_RECORD_CONSTRUCTOR to do the right thing -- in particular,
> > >> delaying the build_constructor call to FINISH_RECORD_CONSTRUCTOR.
> >
> > I think Tom's suggestion is better.
>
> I tried this; the patch is below (along with a few cleanups along the
> way). Clearing TREE_CONSTANT in FINISH_RECORD_CONSTRUCTOR is still
> required to make things go.
I've committed the patch, based on Richi's IRC approval.
-Nathan