[patch] Start removing the old JIT

Sean Silva chisophugis at gmail.com
Sat Jul 26 09:41:12 PDT 2014


On Thu, Jul 24, 2014 at 6:30 PM, Philip Reames <listmail at philipreames.com>
wrote:

>  I fully accept your internal tool argument, but given the documentation
> is more or less "see what google says", making flag migrations obvious
> seems like a helpful thing.  :)
>
>
Actually LLI supposedly has its own man page:
http://llvm.org/docs/CommandGuide/lli.html
I have no idea how up-to-date it is though.

(so do many other of our tools: http://llvm.org/docs/CommandGuide/ )

-- Sean Silva


> Philip
>
>
> On 07/24/2014 03:19 PM, Lang Hames wrote:
>
>  Hi Philip,
>
>  I think adding a short-term warning is a good idea. When the old JIT is
> removed however, the plan is for this option to be removed too. LLI is an
> internal tool, and doesn't guarantee backwards compatibility with flags.
>
>  - Lang.
>
>
> On Thu, Jul 24, 2014 at 2:33 PM, Philip Reames <listmail at philipreames.com>
> wrote:
>
>>
>> On 07/23/2014 09:26 PM, Rafael EspĂ­ndola wrote:
>>
>>> -  cl::opt<bool> UseMCJIT(
>>> -    "use-mcjit", cl::desc("Enable use of the MC-based JIT (if
>>> available)"),
>>> -    cl::init(false));
>>> -
>>>
>> I would suggest that we leave this option in and switch the default.
>>  This keeps command line compatibility for all users.
>>
>> We should add an explicit warning/error for !UseMCJIT though.  If someone
>> is explicitly opting out of MCJIT, we should probably let them know that's
>> not an option any more.
>>
>> This would even make sense as a separate change a few days before
>> anything else.  It would help flush out things which unknowingly or
>> implicitly depend on using the old JIT.
>>
>>
>> Also, where is the documentation change?  That should be part of the very
>> first patch.  :)
>>
>> Philip
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140726/4507c252/attachment.html>


More information about the llvm-commits mailing list