[m-rev.] for review: add a demangled name field to MR_ProcIds
Zoltan Somogyi
zs at cs.mu.OZ.AU
Wed Jan 2 16:44:23 AEDT 2002
On 02-Jan-2002, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> > - That this module be in the library (probably with documentation suppressed
> > as we currently do for e.g. private_builtin), so that we can replace all
> > mangling/demangling code with calls to the code of this library module.
> > This should not just the compiler, the old profiler and the deep profiler,
> > but even include util/mdemangle.c, if we can solve the bootstrapping issue
> > involved.
>
> Why not in the "browser" directory, like browser/name_mangle.m?
At the moment, the browser library is linked into executables iff they are
compiled with debugging. The mangle/demangle module would need to be linked
into several programs (compiler, the profilers, mdemangle) regardless of
whether they are compiled with debugging. It therefore doesn't belong
in the browser library; it should be a separate library. It would therefore
make sense to put it into a directory other than the browser directory.
The obvious alternative to putting it into the standard library directory
is to create a new directory and .a/.so file specifically for it.
Zoltan.
--------------------------------------------------------------------------
mercury-reviews mailing list
post: mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------
More information about the reviews
mailing list