<div dir="ltr"><div dir="ltr">Hi Chris,<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">Thanks for the info. I think we have issues with all of the JIT "prep" work we are currently doing around changing linkage for symbols before jitting. I believe this was done to work with the OrcMCJitReplacement, but it doesn't seem compatible with the new LLJIT. I will look into it further. On second point, we will just turn it off for now.</blockquote><div><br></div><div>Sounds good to me. Hopefully JITLink will evolve ELF support in the not-too-distant future and we will be able to re-enable exception handling.</div><div><br></div><div>Cheers,</div><div>Lang.</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 21, 2019 at 3:58 PM chris boese <<a href="mailto:chris107565@gmail.com">chris107565@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"><div dir="ltr">Hi Lang,<div><br></div><div>Thanks for the info. I think we have issues with all of the JIT "prep" work we are currently doing around changing linkage for symbols before jitting. I believe this was done to work with the OrcMCJitReplacement, but it doesn't seem compatible with the new LLJIT. I will look into it further. On second point, we will just turn it off for now.</div><div><br></div><div>Thanks,<br>Chris</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 19, 2019 at 1:35 PM Lang Hames <<a href="mailto:lhames@gmail.com" target="_blank">lhames@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"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi Chris,<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">1. Using the addIRModule method for adding new global variables, I'm having to set them to external linkage, so that they can be found during lookup. One issue with this, is I have to manually loop through all the underlying globals/functions and change their linkage to external as well. If I don't do that, I'll get a JIT Session Error about not being able to find the underlying globals/functions. Is there an easy way to set all of the underlying globals/functions to external linkage as well? </blockquote><div><br></div><div>There is no "change all linkages" convenience function, you just have to loop over all the global values and set them individually. If you're doing that though there's likely a design issue somewhere else: In general languages will define which symbols get external linkage and which do not, and the front-end should set up the linkages for you: you shouldn't need to modify linkages in order to add something to the JIT. E.g. If you have the following in C:</div><div><br></div><div><font face="monospace">static void foo(void) { }</font></div><div><span style="font-family:monospace">void bar(void) { foo(); }</span><br></div><div><br></div><div>then clang will translate that to:</div><div><br></div><div><div><font face="monospace">define internal void @foo() { ... }</font></div><div><font face="monospace">define void @bar() { ... }</font></div></div><div> </div><div>This IR, with <font face="monospace">foo</font> having internal linkage and <font face="monospace">bar</font> external, can be added to a JITDylib as-is and will result in one visible definition: bar. (ORC can handle internal symbols just fine, they just don't appear in the public interface for the JITDylib because that's what internal linkage means: visible only from within the module).</div><div><br></div><div>The two main exceptions to the "you shouldn't need to promote linkage yourself" rule are:</div><div><br></div><div>(1) Debuggers / REPLs, where you want to give a user the ability to call directly into what would usually be an internal function. In that case you should only need to promote the linkage on the internal symbols that you want the user to be able to access. If that's everything, then promote everything. If that's only certain functions/variables then just promote those functions/variables.</div><div><br></div><div>(2) Manually breaking up modules: If you want to perform more fine grained or parallel compilation you may choose to break up large modules. In that case you need to promote internal symbols to external-hidden symbols so that they can still be resolved across the new module boundaries. You will usually have to rename them to avoid clashes too. If you're using LLLazyJIT though you don't need to worry: all the symbol promoting and renaming required for lazy compilation is taken care of internally.</div><div><br></div><div>So: Why is your front-end producing internal definitions for symbols that you want to be able to call in the JIT? Should your front-end be producing external definitions?</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">2. We've had this problem ever since we switched to OrcMCJitReplacement. When jitting code on linux, we have to comment the body of registerEHFrames(). If we don't, then any exceptions that are thrown in jitted code will result in a segfault. Do you know what issue we might be running into here? RuntimeDyIdELF::registerEHFrames() is commented out for new LLJIT. OrcMCJitReplacement::registerEHFrames when using OrcMCJitReplacement.</blockquote><div><br></div><div>registerEHFrames is supposed to register the __eh_frames section with libunwind. I know that RuntimeDyldMachO's __eh_frame support was weak, and would often result in registering broken frames. I don't know RuntimeDyldELF's __eh_frame support code well, but from a glance I suspect it suffers the same problems. That is almost certainly why your code is segfaulting when exceptions are thrown: the JIT is registering broken frames.</div><div><br></div><div>The poor quality of RuntimeDyld's exception handling is why I turned it off by default. The new JIT linker (JTILink) has much better eh_frame support, but unfortunately there is no ELF JITLink implementation yet.</div><div><br></div><div>The upshot of all this is that at the moment it's not safe in general to throw exceptions from/through JIT'd code on ELF. I'm hoping that JITLink ELF variants will solve this eventually, but there's no clear timeline for that work. </div><div><br></div><div>-- Lang.</div><div><br></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 19, 2019 at 11:12 AM chris boese <<a href="mailto:chris107565@gmail.com" target="_blank">chris107565@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"><div dir="ltr">Hi Lang,<div><br></div><div>Had a couple more questions I thought I'd run by you. </div><div><br></div><div>1. Using the addIRModule method for adding new global variables, I'm having to set them to external linkage, so that they can be found during lookup. One issue with this, is I have to manually loop through all the underlying globals/functions and change their linkage to external as well. If I don't do that, I'll get a JIT Session Error about not being able to find the underlying globals/functions. Is there an easy way to set all of the underlying globals/functions to external linkage as well? </div><div><br></div><div>2. We've had this problem ever since we switched to OrcMCJitReplacement. When jitting code on linux, we have to comment the body of registerEHFrames(). If we don't, then any exceptions that are thrown in jitted code will result in a segfault. Do you know what issue we might be running into here? RuntimeDyIdELF::registerEHFrames() is commented out for new LLJIT. OrcMCJitReplacement::registerEHFrames when using OrcMCJitReplacement.</div><div><br></div><div>Thanks,</div><div>Chris</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 15, 2019 at 9:22 AM chris boese <<a href="mailto:chris107565@gmail.com" target="_blank">chris107565@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"><div dir="ltr">Hi Lang,<div><br></div><div>So that was my issue. Putting modules on the same ThreadSafeContext was causing a deadlock. Our current version of LLVM does not have r367686 patch. I resolved by creating a separate ThreadSafeContext, and now I'm able to add/find globals by using the addLazyIRModule function.</div><div><br></div><div>Thank you so much for your help. For now I believe I'm unblocked. I will reach out again if encountering any more issues.</div><div><br></div><div>Thanks,</div><div>Chris</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 13, 2019 at 2:49 PM Lang Hames <<a href="mailto:lhames@gmail.com" target="_blank">lhames@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"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi Chris,<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">Thanks for clarifying that. That makes sense.<br></blockquote><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"> </blockquote><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">So I apologize. It was not an error I got on the AddLazyIRModule. The issue is actually after I call AddLazyIRModule(). Performing a lookup on that global variable causes a hang. I've stepped through the debugger, and it seems to hang inside Core.cpp, ExecutionSession::lookup(). </blockquote><div><br></div><div>Are you (a) trying to JIT concurrently and (b) loading all modules on the same ThreadSafeContext? Notionally you should be able to do both, but in my experience bugs in this area are the most frequent source of deadlocks in lookup.</div><div><br></div><div>Are you able to get a backtrace for each thread in the debugger while the lookup is deadlocked? If you're using LLDB you can use "thread list" to list the threads, "thread select N" to switch to each, and "bt" to get the backtrace. I am not sure what the equivalent commands in GDB are. Either way, the lookup thread is the least interesting one: I expect to see some other thread waiting on a context lock, and the thing we *really* want to know is who owns the lock and how did they end up blocking while holding it.</div><div><br></div><div>Are you able to share any of your code / test cases?</div><div><br></div><div>Cheers,</div><div>Lang.</div><div><br></div><div>Prior to r367686, ORCv2 took out a lock on the context when a module was emitted and did not relinquish it until that module had been fully emitted. The problem with that is that if the module being emitted has dependencies on other modules living on the same </div><div></div><div><br></div><div>On Tue, Aug 13, 2019 at 2:09 PM chris boese <<a href="mailto:chris107565@gmail.com" target="_blank">chris107565@gmail.com</a>> wrote:<br></div></div><div class="gmail_quote"><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"><div dir="ltr">Hi Lang,<div><br></div><div>Thanks for clarifying that. That makes sense.</div><div><br></div><div>So I apologize. It was not an error I got on the AddLazyIRModule. The issue is actually after I call AddLazyIRModule(). Performing a lookup on that global variable causes a hang. I've stepped through the debugger, and it seems to hang inside Core.cpp, ExecutionSession::lookup(). </div><div><br></div><div>Thanks,</div><div>Chris</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 12, 2019 at 2:33 PM Lang Hames <<a href="mailto:lhames@gmail.com" target="_blank">lhames@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"><div dir="ltr"><div dir="ltr"><div>Hi Chris,</div><div> </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">Thanks so much for the quick help. I was able to find them after changing the linkage. Does this new orc JIT follow different standards than the MCJitReplacement? We used to set a lot of our functions/globals to internal linkage before doing a lookup.</blockquote><div><br></div><div>They are different: ORCv2 follows the static linker's lookup rules, whereas MCJITReplacement mimicked MCJIT's rules which were never clearly spelled out, but in practice allowed you to search for internal symbols.</div><div><br></div><div>The advantage of the new scheme is that if you are compiling/running something on the command line. E.g.:</div><div><br></div><div>$ llc -filetype=obj -o extra.o extra.ll</div><div>$ llc -filetype=obj -o prog.o prog.ll</div><div>$ clang -o prog prog.o extra.o</div><div>$ ./prog</div><div><br></div><div>Then you can add the same modules to the JIT and expect the resulting JIT'd code to behave the same as the statically compiled code. E.g. (omitting some error handling for clarity):</div><div><br></div><div>J->addIRModule(loadModule("extra.ll"));</div><div>J->addIRModule(loadModule("main.ll"));</div><div>auto Main = (int(*)())J->lookup("main").getAddress();</div><div>Main();</div><div> <br></div><div>If you are generating code specifically for the JIT it is often easiest to just use external linkage and default visibility for all symbols. However, for people who want to hook up an existing front-end, or to use more advanced features in their JIT (duplicate symbol names with internal linkage, symbols with weak linkage, hidden visibility, etc.) this scheme has the nice property that "What does linker-feature X do in the JIT?" is more-or-less equivalent to "What does linker-feature X do in a regular build". The answer to the latter question is generally more widely known, and is much easier to test from the command line.<br></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">When adding a new module with a single global variable in it, I get an error during the addLazyIRModule.</blockquote><div><br></div><div>Are you hitting an assertion, or is addLazyIRModule returning a non-success value? If the latter, what is the error message?</div><div>(You can find this by calling 'logAllUnhandledErrors(std::move(Err), errs(), "")', or, in tool code, using the ExitOnError utility: <a href="http://llvm.org/docs/ProgrammersManual.html#using-exitonerror-to-simplify-tool-code" target="_blank">http://llvm.org/docs/ProgrammersManual.html#using-exitonerror-to-simplify-tool-code</a>).</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">Right now we are treating the module as one big single module, and when we go to lookup a symbol, I believe we are jitting the entire thing. Once we make the switch to LLLazyJIT, we want to start splitting up our modules.</blockquote><div><br></div><div>If you are using MCJITReplacement you are definitely JITing the whole module up-front, and in that case lazy compilation may help reduce your compile time. I'd love to hear how it works for you!</div><div><br></div><div>Cheers,</div><div>Lang.</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 12, 2019 at 10:26 AM David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@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"><div dir="ltr"><a class="gmail_plusreply" id="gmail-m_1458841413225016288gmail-m_-4470279675095974576gmail-m_-8643115361749930254gmail-m_-6973040127398969157gmail-m_9046622280019594413gmail-m_1203181100948991706gmail-m_3624009229622175665gmail-m_7363326224696822258gmail-m_-6815707226106742783plusReplyChip-0" href="mailto:lhames@gmail.com" target="_blank">+Lang Hames</a>, Admiral of the Orcish Fleet (Lang's the primary developer/creator of the ORC JIT & might have some answers to your question)</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 8, 2019 at 4:15 PM chris boese via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</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"><div dir="ltr"><div>We are trying to switch to the new orc v2 lljit engine at my work with hopes of parallel jitting. We are switching from the ExecutionEngine w/ OrcMCJitReplacement. We are having a hard time with global variables. We have found a way to create/emit new globals during jitting by using the old ExecutionEngine::getOrEmitGlobalVariable. Is there an easier way to do this with the new jit engine? We were hoping to create a new ThreadSafeModule, add a GlobalVariable, and then add the module to the JIT engine through addLazyIRModule(), but this doesn't seem to work. </div><div><br></div><div>2nd question; Is the lookup function supposed to work with global variables by name? Looking through the CompileOnDemandLayer it seems to change the names of globals to "__orc_lcl.<name><id>". Does this affect the lookup methods? I noticed that all of the globals we are looking for are tagged with Internal linkage and end up getting emitted through extractSubModule().</div><div><br></div><div>Any help is appreciated.</div></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>
</blockquote></div>
</blockquote></div>
</blockquote></div></div></div></div></div>
</blockquote></div>
</blockquote></div>
</blockquote></div></div></div></div></div></div></div>
</blockquote></div>
</blockquote></div>