[patch] Start removing the old JIT

[Philip Reames]

On 07/26/2014 09:41 AM, Sean Silva wrote:
>
>
>
> On Thu, Jul 24, 2014 at 6:30 PM, Philip Reames 
> <listmail at philipreames.com <mailto: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.
Given this page doesn't even *mention* mcjit, not very.

We should fix that.
>
> (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 <mailto: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 <mailto: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 <mailto: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/20140727/08372b27/attachment.html>