[llvm-dev] Change lli default JIT engine to Orc (original) (raw)

Lang Hames via llvm-dev llvm-dev at lists.llvm.org
Sat Mar 20 11:04:30 PDT 2021

Hi Stefan,

I agree that this change is overdue. Thank you very much for working on it. :)

While I think this change is overdue, there is no rush to land it. I am

happy to discuss details in case you have any concerns.

I have no concerns with the intended change. Once you have left a few days for others to comment, and assuming there are no objections, I would go ahead and land this. I suspect you will see some extra failures when this hits the builders, but that's to be expected -- you may find that we need a few rounds of applying and reverting this before all the kinks can be worked out. Landing early gives us plenty of time to sort those out before the next release.

Regards, Lang.

On Fri, Mar 19, 2021 at 3:06 AM Stefan Gränitz <stefan.graenitz at gmail.com> wrote:

Hi all

In review https://reviews.llvm.org/D98931 I am proposing to change lli's default JIT engine to Orc. The main intention here is to get people to realize that MCJIT is poorly maintained and that it might be inappropriate for use with new code and tests. In fact, last week the MLIR subproject added some new tests that use lli with MCJIT. I guess this was unintentional; the reviews at least don't seem to discuss the JIT engine. Common JIT tasks are expected to work with Orc just like they did with MCJIT. All such tests run against both engines now: https://reviews.llvm.org/D98931?id=331802 There is a number of tests that require MCJIT-specific behavior (e.g. interpreter mode). These were patched to explicitly pass -jit-kind=mcjit now: https://reviews.llvm.org/D98931?vs=331802&id=331803 While I think this change is overdue, there is no rush to land it. I am happy to discuss details in case you have any concerns. Cheers Stefan -- https://flowcrypt.com/pub/stefan.graenitz@gmail.com

-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20210320/76ffb1b3/attachment.html>

More information about the llvm-dev mailing list