Re: [libstdc++] Make use of runtime demangler

[Franz Sirl <Franz dot Sirl-kernel at lauterbach dot com>]

On Monday 01 April 2002 19:12, Mark Mitchell wrote:
> --On Friday, March 29, 2002 06:29:24 PM -0500 Phil Edwards <phil@jaj.com>
>
> wrote:
> > This builds the new __cxa_demangle'r into the C++ support library.
> >
> > We link to cp-demangle.c and dyn-string.c, then compile them locally into
> > v3. Two important notes:
> >
> > 1) Those two files do not build warning-free because in the v3 build,
> >    -DHAVE_CONFIG_H is not passed (thus the headers for malloc and whatnot
> >    are not included) because we can't give it the config.h from
> > libiberty. I played with building a symlink to libiberty's config.h but I
> > don't know the multilibs scenario well enough.  Help is requested.  For
> > now, we pass -Wno-error in the special build rules.
>
> I believe we need to fix this, or risk problems on platforms where ints
> and pointers do not have the same size.

Oops, nearly forgot about this one, I have a patch for this. It's actually a 
bug in the libstdc++ Makefiles, cause configure generates config.h for 
libstdc++ too. In trying to pass the DEBUG_FLAGS on the right place in the 
compile command, $(DEFS) got left out.

I chose a not so error-prone setup in using $(COMPILE) and $(CXXCOMPILE) and 
explicitly adding @DEBUG_FLAGS@ instead of the handcrufted stuff.

Bootstrapped/regtested on powerpc-linux-gnu and x86-linux-gnu.

Ah, this doesn't cure the warnings for the string functions yet, cause 
configure doesn't check for string.h and I wasn't sure where to add it.

Franz.


	* libsupc++/Makefile.am (LTCOMPILE): Use $(COMPILE).
	(LTCXXCOMPILE): Use $(CXXCOMPILE).
	* src/Makefile.am (LTCXXCOMPILE): Use $(CXXCOMPILE).
	* libsupc++/Makefile.in: Regenerate.
	* src/Makefile.in: Regenerate.

Attachment: gcc-libstdc++-Makefile.patch
Description: Text document