[RFA:] PR37170, extern weak broken, like for gcc.dg/weak/weak-1.c

Hans-Peter Nilsson hans-peter.nilsson@axis.com
Mon Aug 25 12:34:00 GMT 2008


> Date: Mon, 25 Aug 2008 08:23:27 +0100
> From: "Rafael Espindola" <espindola@google.com>

> > I have to retract this patch: it's not altogether wrong, but a
> > bit incomplet.
> 
> If it is just incomplete, shouldn't it be better to commit it and then
> fix the remaining parts?

Because I wasn't really sure.  Now that I have more confidence
in the current patch, I'm just waiting for test-results.  See
the PR.

> Is there something I can do to help?

Yes, there's a related change in revision 138310.  From the
mailing list conversation it looks like it's just papering over
the output_operand machinery not being able to "catch" all
symbol_refs.  This should be fixed by my "take 4" patch.  I'd
just like to verify that with the "original" pair of eyes.

But, all of this seems just a hack anyway, and the
assemble_external calls should be only in the "expand" pass; not
in front-ends nor in the back-end (varasm.c, final.c).  If you
could fix *that*, it'd be nice.

brgds, H-P



More information about the Gcc-patches mailing list