[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