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@cs.uiuc.edu > > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits > _______________________________________________ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
