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/
> 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
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
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
> 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