[MPlayer-advusers] gcc, mplayer and profile (mcount)

[Alban Bedel]

On Wed, 3 Jan 2007 15:30:52 +0100
"Guillaume POIRIER" <poirierg at gmail.com> wrote:

> Hi,
> 
> On 1/3/07, Adam Sulmicki <adam at cfar.umd.edu> wrote:
> >
> > Hello folks,
> >
> >         This is my last post on the subject of mcount.
> >
> >         I have spent a quite bit of time on this to find out
> >         that the results of myserious crashes is the mcount variable.
> >         (with help from Ian Lance Taylor).
> >
> >         I have reported the issue to both gcc and mplayer lists, and
> >         from _BOTH_ lists I got reply that they do not feel it is their
> >         problem and blamed the other package.
> 
> This is interesting. Do you have a link to their reply? I'd like to
> see what they had to say, and if they have an explanation for this.

I would definitly be intersted too. 

 
> [..]
> 
> > Suggested solutions:
> >
> >         * Rename mcount in mplayer/libmenu/menu.c to something else.
> 
> this is simple enough that it should probably done, yeah. (though I
> dislike working around compiler bugs).

I agree, althougth if a patch would have been provided ...

> >
> >         * document mcount in gcc man page
> >
> >         * gcc prints warning.
> >
> >         * do nothing.
> >
> > I can immagine that gcc developers might have intentionally left mcount()
> > visible to user space so that user can replace gcc's mcount() with their
> > own implementation.
> 
> Well, as far as I understand how names clashes are supposed to be
> avoided, the compiler should define and use symbols that start with
> "_", everything else is supposed to be free of previous
> implementation....
> ... but maybe it's just me...

That's what i understood too although i'm not sure if that's not for the
libc only. Anyway if that's the case they should at least use a quiet
longer name and/or make it configurable.

	Albeu