From b67fec3d6f1f1c87cd3d8037701d4681b96bfd85 Mon Sep 17 00:00:00 2001 From: Mehdi Amini Date: Thu, 7 Mar 2024 15:15:56 -0800 Subject: [PATCH] Revert "[Orc] Add NotifyCreated callback for LLJITBuilder (#84175)" This reverts commit f78129e2bbafdd04a71bc09fc44e0797dd08db05. --- llvm/include/llvm/ExecutionEngine/Orc/LLJIT.h | 18 ------------------ 1 file changed, 18 deletions(-) diff --git a/llvm/include/llvm/ExecutionEngine/Orc/LLJIT.h b/llvm/include/llvm/ExecutionEngine/Orc/LLJIT.h index d5682fcaa28b7..76d16e63df281 100644 --- a/llvm/include/llvm/ExecutionEngine/Orc/LLJIT.h +++ b/llvm/include/llvm/ExecutionEngine/Orc/LLJIT.h @@ -311,8 +311,6 @@ class LLJITBuilderState { using PlatformSetupFunction = unique_function(LLJIT &J)>; - using NotifyCreatedFunction = std::function; - std::unique_ptr EPC; std::unique_ptr ES; std::optional JTMB; @@ -323,7 +321,6 @@ class LLJITBuilderState { CompileFunctionCreator CreateCompileFunction; unique_function PrePlatformSetup; PlatformSetupFunction SetUpPlatform; - NotifyCreatedFunction NotifyCreated; unsigned NumCompileThreads = 0; /// Called prior to JIT class construcion to fix up defaults. @@ -444,16 +441,6 @@ class LLJITBuilderSetters { return impl(); } - /// Set up a callback after successful construction of the JIT. - /// - /// This is useful to attach generators to JITDylibs or inject initial symbol - /// definitions. - SetterImpl & - setNotifyCreatedCallback(LLJITBuilderState::NotifyCreatedFunction Callback) { - impl().NotifyCreated = std::move(Callback); - return impl(); - } - /// Set the number of compile threads to use. /// /// If set to zero, compilation will be performed on the execution thread when @@ -487,11 +474,6 @@ class LLJITBuilderSetters { std::unique_ptr J(new JITType(impl(), Err)); if (Err) return std::move(Err); - - if (impl().NotifyCreated) - if (Error Err = impl().NotifyCreated(*J)) - return Err; - return std::move(J); }