Making invalidateInstructionCache automatic in SectionMemoryManager
authorAndrew Kaylor <andrew.kaylor@intel.com>
Wed, 24 Apr 2013 22:39:12 +0000 (22:39 +0000)
committerAndrew Kaylor <andrew.kaylor@intel.com>
Wed, 24 Apr 2013 22:39:12 +0000 (22:39 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@180225 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/ExecutionEngine/SectionMemoryManager.h
lib/ExecutionEngine/MCJIT/SectionMemoryManager.cpp

index ae5004e130c0415010173e59de57d32b1ba4b828..9ac8c92524a0915534fe82401e3f8f951aad4efc 100644 (file)
@@ -87,9 +87,7 @@ public:
   /// explicit cache flush, otherwise JIT code manipulations (like resolved
   /// relocations) will get to the data cache but not to the instruction cache.
   ///
-  /// This method is not called by RuntimeDyld or MCJIT during the load
-  /// process.  Clients may call this function when needed.  See the lli
-  /// tool for example use.
+  /// This method is called from applyPermissions.
   virtual void invalidateInstructionCache();
 
 private:
index fa35acd389aed4fc5e5114a3b7f97952371e4fd1..da931247b6cac1aca9424dc679547ba63c363abb 100644 (file)
@@ -138,6 +138,11 @@ bool SectionMemoryManager::applyPermissions(std::string *ErrMsg)
 
   // Read-write data memory already has the correct permissions
 
+  // Some platforms with separate data cache and instruction cache require
+  // explicit cache flush, otherwise JIT code manipulations (like resolved
+  // relocations) will get to the data cache but not to the instruction cache.
+  invalidateInstructionCache();
+
   return false;
 }