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

2007-05-02 Thread Reid Spencer
Leo, Looks like Chris didn't wait very long before reverting the CBE patch. Please try your patch again. It *must* pass all of llvm/test and llvm-test to be accepted. Thanks, Reid. On Wed, 2 May 2007 21:57:31 -0500 Chris Lattner <[EMAIL PROTECTED]> wrote: > > >Changes in directory llvm/lib/

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

2007-04-12 Thread Chris Lattner
> Provide support for intrinsics that lower themselves to a function > body. > This can happen for intrinsics that are overloaded. In such cases > it is > necessary to emit a function prototype before the body of the function > that calls the intrinsic and to ensure we don't emit it multiple

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

2007-02-03 Thread Chris Lattner
On Jan 26, 2007, at 1:43 PM, Reid Spencer wrote: > 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 sym

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

2007-01-26 Thread 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 glo

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

2007-01-26 Thread Chris Lattner
> 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? -Chris > > --- > Diffs