This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [c++/cgraph] use cgraph in expand_or_defer_fn


On Thu, Sep 11, 2003 at 05:13:37PM +0200, Jan Hubicka wrote:
> I would preffer the cgraph_mark_reachable node to be simple marking
> function and in particular to not do too much of recursion
> (while recursion over nested function depth looks resonable to me,
> recursive DFS search is bit too expensive).

I agree.  That hack wouldn't have gone in; it was just trying
to see if that was the problem.  It quickly proved that the
decl in question was never arriving.

> Also concerning the reachability it looks like your problem is in the
> fact that C++ frontend does not emit functions it don't see to be needed
> via SYMBOL_REFERENCED flag when working in non-unit-at-a-time mode.

Indeed.

> At the moment I believe that when constructor use specialization of
> method that is not used elsewhere, C++ frontend won't synthetize it as
> it is all guarded by DECL_NEEDED_P and we run into troubles.

I think the real fix is to put instantiation under the control
of cgraph as well.  I have a Cunning Plan.  Hopefully it will
be in place by the time you get back.


r~


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]