Stefan Gränitz via llvm-dev
2021-Mar-19 10:06 UTC
[llvm-dev] Change lli default JIT engine to Orc
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 at gmail.com
Lang Hames via llvm-dev
2021-Mar-20 18:04 UTC
[llvm-dev] Change lli default JIT engine to Orc
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 at gmail.com > > >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20210320/76ffb1b3/attachment.html>