[Lldb-commits] [lldb] r182099 - Fixed the build to reflect the API name change in r182085.
Ashok Thirumurthi
ashok.thirumurthi at intel.com
Fri May 17 07:46:59 PDT 2013
Author: athirumu
Date: Fri May 17 09:46:59 2013
New Revision: 182099
URL: http://llvm.org/viewvc/llvm-project?rev=182099&view=rev
Log:
Fixed the build to reflect the API name change in r182085.
- Also added a comment as lldb doesn't flush the instruction cache after dy-load.
Modified:
lldb/trunk/include/lldb/Expression/IRExecutionUnit.h
Modified: lldb/trunk/include/lldb/Expression/IRExecutionUnit.h
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/include/lldb/Expression/IRExecutionUnit.h?rev=182099&r1=182098&r2=182099&view=diff
==============================================================================
--- lldb/trunk/include/lldb/Expression/IRExecutionUnit.h (original)
+++ lldb/trunk/include/lldb/Expression/IRExecutionUnit.h Fri May 17 09:46:59 2013
@@ -336,7 +336,13 @@ private:
/// @return
/// True in case of failure, false in case of success.
//------------------------------------------------------------------
- bool applyPermissions(std::string *ErrMsg) { return false; }
+ virtual bool finalizeMemory(std::string *ErrMsg) {
+ // TODO: Ensure that the instruction cache is flushed because
+ // relocations are updated by dy-load. See:
+ // sys::Memory::InvalidateInstructionCache
+ // llvm::SectionMemoryManager
+ return false;
+ }
//------------------------------------------------------------------
/// Passthrough interface stub
More information about the lldb-commits
mailing list