Skip to content

Commit 3295b88

Browse files
authored
Revert "[mlir][ExecutionEngine] Add support for global constructors and destructors" (llvm#78164)
this is causing test failures on AArch64 linux, hitting the following assert: # | mlir-cpu-runner: /home/culrho01/llvm-project/llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp:519: void llvm::RuntimeDyldELF::resolveAArch64Relocation(const SectionEntry &, uint64_t, uint64_t, uint32_t, int64_t): Assertion `isInt<33>(Result) && "overflow check failed for relocation"' failed. Seeing the same in buildbot as well, e.g. https://lab.llvm.org/buildbot/#/builders/179/builds/9094/steps/12/logs/FAIL__MLIR__sparse_codegen_dim_mlir Reverts llvm#78070
1 parent c5a9e35 commit 3295b88

File tree

2 files changed

+0
-38
lines changed

2 files changed

+0
-38
lines changed

mlir/lib/ExecutionEngine/ExecutionEngine.cpp

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -219,9 +219,6 @@ ExecutionEngine::ExecutionEngine(bool enableObjectDump,
219219
}
220220

221221
ExecutionEngine::~ExecutionEngine() {
222-
// Execute the global destructors from the module being processed.
223-
if (jit)
224-
llvm::consumeError(jit->deinitialize(jit->getMainJITDylib()));
225222
// Run all dynamic library destroy callbacks to prepare for the shutdown.
226223
for (LibraryDestroyFn destroy : destroyFns)
227224
destroy();
@@ -399,9 +396,6 @@ ExecutionEngine::create(Operation *m, const ExecutionEngineOptions &options,
399396
};
400397
engine->registerSymbols(runtimeSymbolMap);
401398

402-
// Execute the global constructors from the module being processed.
403-
cantFail(engine->jit->initialize(engine->jit->getMainJITDylib()));
404-
405399
return std::move(engine);
406400
}
407401

mlir/test/mlir-cpu-runner/global-constructors.mlir

Lines changed: 0 additions & 32 deletions
This file was deleted.

0 commit comments

Comments
 (0)