<div dir="ltr"><div dir="ltr">Hi Stefan,<div><br></div><div>I agree that this change is overdue. Thank you very much for working on it. :)</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">While I think this change is overdue, there is no rush to land it. I am<br>happy to discuss details in case you have any concerns.</blockquote><div><br></div><div>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.</div><div><br></div><div>Regards,</div><div>Lang.</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 19, 2021 at 3:06 AM Stefan Gränitz <<a href="mailto:stefan.graenitz@gmail.com">stefan.graenitz@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Hi all<br>
<br>
In review <a href="https://reviews.llvm.org/D98931" rel="noreferrer" target="_blank">https://reviews.llvm.org/D98931</a> I am proposing to change lli's<br>
default JIT engine to Orc. The main intention here is to get people to<br>
realize that MCJIT is poorly maintained and that it might be<br>
inappropriate for use with new code and tests. In fact, last week the<br>
MLIR subproject added some new tests that use lli with MCJIT. I guess<br>
this was unintentional; the reviews at least don't seem to discuss the<br>
JIT engine.<br>
<br>
Common JIT tasks are expected to work with Orc just like they did with<br>
MCJIT. All such tests run against both engines now:<br>
<a href="https://reviews.llvm.org/D98931?id=331802" rel="noreferrer" target="_blank">https://reviews.llvm.org/D98931?id=331802</a><br>
<br>
There is a number of tests that require MCJIT-specific behavior (e.g.<br>
interpreter mode). These were patched to explicitly pass -jit-kind=mcjit<br>
now:<br>
<a href="https://reviews.llvm.org/D98931?vs=331802&id=331803" rel="noreferrer" target="_blank">https://reviews.llvm.org/D98931?vs=331802&id=331803</a><br>
<br>
While I think this change is overdue, there is no rush to land it. I am<br>
happy to discuss details in case you have any concerns.<br>
<br>
Cheers<br>
Stefan<br>
<br>
-- <br>
<a href="https://flowcrypt.com/pub/stefan.graenitz@gmail.com" rel="noreferrer" target="_blank">https://flowcrypt.com/pub/stefan.graenitz@gmail.com</a><br>
<br>
<br>
</blockquote></div>