[Valgrind-developers] Valgrind: r15160 - /trunk/coregrind/m_debuginfo/readdwarf.c

[<sv...@va...>]

Author: sewardj
Date: Thu Apr 30 12:45:13 2015
New Revision: 15160

Log:
DW_CFA_def_cfa_expression: don't push the CFA on the stack before
evaluation starts.  For DW_CFA_val_expression and DW_CFA_expression
doing so is correct, but not for DW_CFA_def_cfa_expression.

Modified:
    trunk/coregrind/m_debuginfo/readdwarf.c

Modified: trunk/coregrind/m_debuginfo/readdwarf.c
==============================================================================
--- trunk/coregrind/m_debuginfo/readdwarf.c (original)
+++ trunk/coregrind/m_debuginfo/readdwarf.c Thu Apr 30 12:45:13 2015
@@ -3311,7 +3311,7 @@
             VG_(printf)("  DW_CFA_def_cfa_expression (");
          /* Convert the expression into a dag rooted at ctx->exprs index j,
             or fail. */
-         j = dwarfexpr_to_dag ( ctx, expr, len, True/*push CFA at start*/, 
+         j = dwarfexpr_to_dag ( ctx, expr, len, False/*!push CFA at start*/, 
                                 di->ddump_frames);
          if (di->ddump_frames)
             VG_(printf)(")\n");