[llvm] r191938 - Adding support and tests for multiple module handling in lli

[Kaylor, Andrew]

Oops.  I missed that somehow.  The good news is those calls aren't necessary.  I'll get rid of that today.

-----Original Message-----
From: Alp Toker [mailto:alp at nuanti.com] 
Sent: Tuesday, October 08, 2013 6:04 AM
To: Kaylor, Andrew
Cc: Richard Smith; llvm-commits
Subject: Re: [llvm] r191938 - Adding support and tests for multiple module handling in lli

Did this get looked at?


On 04/10/2013 22:28, Richard Smith wrote:
>
>     +// FIXME: Don't commit like this.  I don't think these calls are
>     necessary.
>
>
> !!!
>  
>
>     +#if 0
>          // Trigger compilation.
>          EE->generateCodeForModule(Mod);
>
>     -    // Layout the target memory.
>     -    layoutRemoteTargetMemory(Target.get(), MM);
>     +    // Get everything ready to execute.
>     +    EE->finalizeModule(Mod);
>     +#endif
>

-- 
http://www.nuanti.com
the browser experts