[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