From 210539ebc466521e41e69b119649d59cc721b006 Mon Sep 17 00:00:00 2001 From: Anton Korobeynikov Date: Tue, 17 Jun 2008 17:30:05 +0000 Subject: [PATCH] Provide generic hooks for icache invalidation. Add PPC implementation. Patch by Gary Benson! git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@52418 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/Target/TargetJITInfo.h | 5 +++++ lib/ExecutionEngine/JIT/JITEmitter.cpp | 16 +--------------- lib/Target/PowerPC/PPCCodeEmitter.cpp | 4 ---- lib/Target/PowerPC/PPCJITInfo.cpp | 15 +++++---------- lib/Target/PowerPC/PPCJITInfo.h | 5 +++++ 5 files changed, 16 insertions(+), 29 deletions(-) diff --git a/include/llvm/Target/TargetJITInfo.h b/include/llvm/Target/TargetJITInfo.h index 6e1eb4396c6..293e0d38406 100644 --- a/include/llvm/Target/TargetJITInfo.h +++ b/include/llvm/Target/TargetJITInfo.h @@ -94,6 +94,11 @@ namespace llvm { assert(NumRelocs == 0 && "This target does not have relocations!"); } + /// InvalidateInstructionCache - Before the JIT can run a block of code + // that has been emitted it must invalidate the instruction cache on some + // platforms. + virtual void InvalidateInstructionCache(const void *Addr, unsigned len) {} + /// needsGOT - Allows a target to specify that it would like the // JIT to manage a GOT for it. bool needsGOT() const { return useGOT; } diff --git a/lib/ExecutionEngine/JIT/JITEmitter.cpp b/lib/ExecutionEngine/JIT/JITEmitter.cpp index 422022af9ca..c55bc75560c 100644 --- a/lib/ExecutionEngine/JIT/JITEmitter.cpp +++ b/lib/ExecutionEngine/JIT/JITEmitter.cpp @@ -145,20 +145,6 @@ namespace { JITResolver *JITResolver::TheJITResolver = 0; -#if (defined(__POWERPC__) || defined (__ppc__) || defined(_POWER)) && \ - defined(__APPLE__) -extern "C" void sys_icache_invalidate(const void *Addr, size_t len); -#endif - -/// synchronizeICache - On some targets, the JIT emitted code must be -/// explicitly refetched to ensure correct execution. -static void synchronizeICache(const void *Addr, size_t len) { -#if (defined(__POWERPC__) || defined (__ppc__) || defined(_POWER)) && \ - defined(__APPLE__) - sys_icache_invalidate(Addr, len); -#endif -} - /// getFunctionStub - This returns a pointer to a function stub, creating /// one on demand as needed. void *JITResolver::getFunctionStub(Function *F) { @@ -756,7 +742,7 @@ bool JITEmitter::finishFunction(MachineFunction &F) { } // Invalidate the icache if necessary. - synchronizeICache(FnStart, FnEnd-FnStart); + TheJIT->getJITInfo().InvalidateInstructionCache(FnStart, FnEnd-FnStart); // Add it to the JIT symbol table if the host wants it. AddFunctionToSymbolTable(F.getFunction()->getNameStart(), diff --git a/lib/Target/PowerPC/PPCCodeEmitter.cpp b/lib/Target/PowerPC/PPCCodeEmitter.cpp index c0621a074f3..51672be0491 100644 --- a/lib/Target/PowerPC/PPCCodeEmitter.cpp +++ b/lib/Target/PowerPC/PPCCodeEmitter.cpp @@ -80,10 +80,6 @@ FunctionPass *llvm::createPPCCodeEmitterPass(PPCTargetMachine &TM, return new PPCCodeEmitter(TM, MCE); } -#ifdef __APPLE__ -extern "C" void sys_icache_invalidate(const void *Addr, size_t len); -#endif - bool PPCCodeEmitter::runOnMachineFunction(MachineFunction &MF) { assert((MF.getTarget().getRelocationModel() != Reloc::Default || MF.getTarget().getRelocationModel() != Reloc::Static) && diff --git a/lib/Target/PowerPC/PPCJITInfo.cpp b/lib/Target/PowerPC/PPCJITInfo.cpp index db36881ae53..72001ce334b 100644 --- a/lib/Target/PowerPC/PPCJITInfo.cpp +++ b/lib/Target/PowerPC/PPCJITInfo.cpp @@ -330,12 +330,9 @@ defined(__APPLE__) extern "C" void sys_icache_invalidate(const void *Addr, size_t len); #endif -/// SyncICache - On PPC, the JIT emitted code must be explicitly refetched to -/// ensure correct execution. -static void SyncICache(const void *Addr, size_t len) { -#if defined(__POWERPC__) || defined (__ppc__) || defined(_POWER) - -#ifdef __APPLE__ +void PPCJITInfo::InvalidateInstructionCache(const void *Addr, unsigned len) { +#if (defined(__POWERPC__) || defined (__ppc__) || defined(_POWER)) && \ +defined(__APPLE__) sys_icache_invalidate(Addr, len); #elif defined(__GNUC__) const size_t LineSize = 32; @@ -352,8 +349,6 @@ static void SyncICache(const void *Addr, size_t len) { asm volatile("icbi 0, %0" : : "r"(Line)); asm volatile("isync"); #endif - -#endif } void *PPCJITInfo::emitFunctionStub(const Function* F, void *Fn, @@ -372,7 +367,7 @@ void *PPCJITInfo::emitFunctionStub(const Function* F, void *Fn, MCE.emitWordBE(0); MCE.emitWordBE(0); EmitBranchToAt(Addr, (intptr_t)Fn, false, is64Bit); - SyncICache((void*)Addr, 7*4); + InvalidateInstructionCache((void*)Addr, 7*4); return MCE.finishFunctionStub(F); } @@ -400,7 +395,7 @@ void *PPCJITInfo::emitFunctionStub(const Function* F, void *Fn, MCE.emitWordBE(0); MCE.emitWordBE(0); EmitBranchToAt(BranchAddr, (intptr_t)Fn, true, is64Bit); - SyncICache((void*)Addr, 10*4); + InvalidateInstructionCache((void*)Addr, 10*4); return MCE.finishFunctionStub(F); } diff --git a/lib/Target/PowerPC/PPCJITInfo.h b/lib/Target/PowerPC/PPCJITInfo.h index c93a84aca05..b7e6ff56ab1 100644 --- a/lib/Target/PowerPC/PPCJITInfo.h +++ b/lib/Target/PowerPC/PPCJITInfo.h @@ -41,6 +41,11 @@ namespace llvm { /// code. /// virtual void replaceMachineCodeForFunction(void *Old, void *New); + + /// InvalidateInstructionCache - Before the JIT can run a block of code + // that has been emitted it must invalidate the instruction cache on some + // platforms. + virtual void InvalidateInstructionCache(const void *Addr, unsigned len); }; } -- 2.34.1