[llvm-commits] [llvm-gcc-4.2] r53130 - /llvm-gcc-4.2/trunk/gcc/cgraphunit.c

[Eric Christopher]

On Jul 5, 2008, at 10:02 AM, Chris Lattner wrote:

> Eric, this is only a change for C++ and it is a change back to the
> previous behavior.  C++ also "suffers" from inline functions not being
> being emitted if not used, so it seems reasonable for static functions
> to behave the same way.  This obviously follows some user's
> expectations.

Dale's last message was pretty convincing. I was just letting it  
percolate
a bit.

> The reason this matters is that real libstdc++ headers have static
> functions and they obviously don't expect them to be emitted (they
> really are not debugging hooks).  Emitting them really does bloat
> executables and significantly slow down the compiler (extra codegen +
> tons of extra debug info), so this really does impact real users.
> Emitted dead functions on the basis that they might be debug hooks is
> a theoretical benefit to theoretical users who have a better solution
> to their problem anyway.

*nod*

OK. I'll stop arguing :)

-eric