[llvm-commits] CVS: llvm/lib/Target/CBackend/CBackend.cpp

[Reid Spencer]

On Fri, 2007-01-26 at 13:33 -0800, Chris Lattner wrote:
> > For PR645: http://llvm.org/PR645 :
> > Remove the Function::renameLocalSymbols function as it is no longer
> > needed.
> 
> I'm not sure this one is safe.  Does the CBE prefix local symbols  
> with something to make them distinct from global symbols in the  
> generated .cbe.c code?

It does so much name mangling, I'm just not sure. I'll look into it.

> 
> -Chris
> 
> >
> > ---
> > Diffs of the changes:  (+0 -3)
> >
> >  CBackend.cpp |    3 ---
> >  1 files changed, 3 deletions(-)
> >
> >
> > Index: llvm/lib/Target/CBackend/CBackend.cpp
> > diff -u llvm/lib/Target/CBackend/CBackend.cpp:1.319 llvm/lib/Target/ 
> > CBackend/CBackend.cpp:1.320
> > --- llvm/lib/Target/CBackend/CBackend.cpp:1.319	Tue Jan 16 12:02:45  
> > 2007
> > +++ llvm/lib/Target/CBackend/CBackend.cpp	Fri Jan 26 02:01:30 2007
> > @@ -99,9 +99,6 @@
> >        // Output all floating point constants that cannot be  
> > printed accurately.
> >        printFloatingPointConstants(F);
> >
> > -      // Ensure that no local symbols conflict with global symbols.
> > -      F.renameLocalSymbols();
> > -
> >        printFunction(F);
> >        FPConstantMap.clear();
> >        return false;
> >
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>