[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