[llvm] [JIT] Fix crash in unit tests (PR #113492)

Lang Hames via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 23 14:21:32 PDT 2024


================
@@ -59,12 +59,13 @@ class JITLinkRedirectableSymbolManager : public RedirectableSymbolManager,
             ObjLinkingLayer.getExecutionSession().getTargetTriple())),
         PtrJumpStubCreator(jitlink::getPointerJumpStubCreator(
             ObjLinkingLayer.getExecutionSession().getTargetTriple())) {
+    ErrorAsOutParameter _(&Err);
+    ObjLinkingLayer.getExecutionSession().registerResourceManager(*this);
     if (!AnonymousPtrCreator || !PtrJumpStubCreator)
       Err = make_error<StringError>("Architecture not supported",
                                     inconvertibleErrorCode());
     if (Err)
       return;
-    ObjLinkingLayer.getExecutionSession().registerResourceManager(*this);
----------------
lhames wrote:

Is it necessary to move the
```c++
ObjLinkingLayer.getExecutionSession().registerResourceManager(*this);
```
line up here, or can that stay where it was?

https://github.com/llvm/llvm-project/pull/113492


More information about the llvm-commits mailing list