[llvm-commits] [llvm] r85295 - in /llvm/trunk: docs/tutorial/LangImpl4.html docs/tutorial/OCamlLangImpl4.html include/llvm/ExecutionEngine/ExecutionEngine.h lib/ExecutionEngine/ExecutionEngine.cpp lib/ExecutionEngine/JIT/JIT.cpp lib/ExecutionEngi

[Jeffrey Yasskin]

On Tue, Oct 27, 2009 at 1:54 PM, Evan Cheng <evan.cheng at apple.com> wrote:
>
> On Oct 27, 2009, at 1:30 PM, Jeffrey Yasskin wrote:
>
>> Author: jyasskin
>> Date: Tue Oct 27 15:30:28 2009
>> New Revision: 85295
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=85295&view=rev
>> Log:
>> Change the JIT to compile eagerly by default as agreed in
>> http://llvm.org/PR5184, and beef up the comments to describe what both
>> options
>> do and the risks of lazy compilation in the presence of threads.
>
> Hi Jeffrey,
>
> In the future I'd prefer API changes be agreed upon by the greater
> community, not just in a bugzilla report.

Sorry about that. Do you want me to revert this until we can ping llvmdev?

> Lazy compilation is being used by some important clients. They will be
> caught off guard by this change. Does this change lli default behavior?

No, it doesn't change lli's default, although that was an accident on
my part (maybe a fortunate accident). I did ping llvmdev last week
asking people who use the lazy JIT to look at the bug report, but I
can see how people who don't use threads with the JIT would think it
didn't apply to them.