From d90eb5527c869d6f478b6706e3c5642feb9e6088 Mon Sep 17 00:00:00 2001 From: Lang Hames Date: Fri, 21 Nov 2014 01:57:09 +0000 Subject: [PATCH] [MCJIT] Remove JITEventListener::NotifyFreeingMachineCode. This method is dead now that the old JIT has been removed. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@222494 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/ExecutionEngine/JITEventListener.h | 10 ---------- .../IntelJITEvents/IntelJITEventListener.cpp | 10 ---------- .../OProfileJIT/OProfileJITEventListener.cpp | 12 ------------ 3 files changed, 32 deletions(-) diff --git a/include/llvm/ExecutionEngine/JITEventListener.h b/include/llvm/ExecutionEngine/JITEventListener.h index 02350b6a950..cef3aa27d5c 100644 --- a/include/llvm/ExecutionEngine/JITEventListener.h +++ b/include/llvm/ExecutionEngine/JITEventListener.h @@ -59,16 +59,6 @@ public: JITEventListener() {} virtual ~JITEventListener(); - /// NotifyFreeingMachineCode - Called from freeMachineCodeForFunction(), after - /// the global mapping is removed, but before the machine code is returned to - /// the allocator. - /// - /// OldPtr is the address of the machine code and will be the same as the Code - /// parameter to a previous NotifyFunctionEmitted call. The Function passed - /// to NotifyFunctionEmitted may have been destroyed by the time of the - /// matching NotifyFreeingMachineCode call. - virtual void NotifyFreeingMachineCode(void *) {} - /// NotifyObjectEmitted - Called after an object has been successfully /// emitted to memory. NotifyFunctionEmitted will not be called for /// individual functions in the object. diff --git a/lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp b/lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp index ec9414750bd..b23ca88baf1 100644 --- a/lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp +++ b/lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp @@ -57,8 +57,6 @@ public: ~IntelJITEventListener() { } - virtual void NotifyFreeingMachineCode(void *OldPtr); - virtual void NotifyObjectEmitted(const ObjectImage &Obj); virtual void NotifyFreeingObject(const ObjectImage &Obj); @@ -97,14 +95,6 @@ static iJIT_Method_Load FunctionDescToIntelJITFormat( return Result; } -void IntelJITEventListener::NotifyFreeingMachineCode(void *FnStart) { - MethodIDMap::iterator I = MethodIDs.find(FnStart); - if (I != MethodIDs.end()) { - Wrapper->iJIT_NotifyEvent(iJVM_EVENT_TYPE_METHOD_UNLOAD_START, &I->second); - MethodIDs.erase(I); - } -} - void IntelJITEventListener::NotifyObjectEmitted(const ObjectImage &Obj) { // Get the address of the object image for use as a unique identifier const void* ObjData = Obj.getData().data(); diff --git a/lib/ExecutionEngine/OProfileJIT/OProfileJITEventListener.cpp b/lib/ExecutionEngine/OProfileJIT/OProfileJITEventListener.cpp index 1fd24f12f40..5a8ccb66e52 100644 --- a/lib/ExecutionEngine/OProfileJIT/OProfileJITEventListener.cpp +++ b/lib/ExecutionEngine/OProfileJIT/OProfileJITEventListener.cpp @@ -49,8 +49,6 @@ public: ~OProfileJITEventListener(); - virtual void NotifyFreeingMachineCode(void *OldPtr); - virtual void NotifyObjectEmitted(const ObjectImage &Obj); virtual void NotifyFreeingObject(const ObjectImage &Obj); @@ -77,16 +75,6 @@ OProfileJITEventListener::~OProfileJITEventListener() { } } -// Removes the being-deleted function from the symbol table. -void OProfileJITEventListener::NotifyFreeingMachineCode(void *FnStart) { - assert(FnStart && "Invalid function pointer"); - if (Wrapper.op_unload_native_code(reinterpret_cast(FnStart)) == -1) { - DEBUG(dbgs() - << "Failed to tell OProfile about unload of native function at " - << FnStart << "\n"); - } -} - void OProfileJITEventListener::NotifyObjectEmitted(const ObjectImage &Obj) { if (!Wrapper.isAgentAvailable()) { return; -- 2.34.1