[Lldb-commits] [PATCH] Fix handling of DW_CFA_restore_state

Tamas Berghammer tberghammer at google.com
Tue Jun 30 10:28:53 PDT 2015


Hi jasonmolenda,

Fix handling of DW_CFA_restore_state

The CFA offset shouldn't be restored to the saved value in case of a DW_CFA_restore_state opcode.

http://reviews.llvm.org/D10843

Files:
  source/Symbol/DWARFCallFrameInfo.cpp

Index: source/Symbol/DWARFCallFrameInfo.cpp
===================================================================
--- source/Symbol/DWARFCallFrameInfo.cpp
+++ source/Symbol/DWARFCallFrameInfo.cpp
@@ -743,8 +743,10 @@
                     // useful for compilers that move epilogue code into the body of a
                     // function.)
                     {
+                        lldb::addr_t offset = row->GetOffset ();
                         row = stack.back ();
                         stack.pop_back ();
+                        row->SetOffset (offset);
                     }
                     break;

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D10843.28797.patch
Type: text/x-patch
Size: 620 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150630/ed2213d2/attachment.bin>


More information about the lldb-commits mailing list