[patch] Start removing the old JIT

Philip Reames listmail at philipreames.com
Thu Jul 24 14:33:28 PDT 2014


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




More information about the llvm-commits mailing list