commit 06b83ee33ad1f4b434d011d164c212be7c531cb6
author Eugene Tarassov <eugene.tarassov@xilinx.com> Fri Mar 20 18:48:27 2015 -0700
committer Eugene Tarassov <eugene.tarassov@xilinx.com> Fri Mar 20 18:48:27 2015 -0700
tree 2e28a982c6824bd37aaea53c4a417f74a6c0db83
parent 18a25da0dd49501b9e1d68c2bcd32d655e559e3c

Bug 461618 - TCF Debug needs to improve support for inline methods

agent/tcf/services/runctrl.c

diff --git a/agent/tcf/services/runctrl.c b/agent/tcf/services/runctrl.c
index b8367c8..f1d6e92 100644
--- a/agent/tcf/services/runctrl.c
+++ b/agent/tcf/services/runctrl.c
@@ -82,6 +82,7 @@
     ContextAddress step_bp_addr;
     BreakpointInfo * step_bp_info;
     int step_inlined;
+    int step_chk_inline_frame;
     CodeArea * step_code_area;
     ErrorReport * step_error;
     const char * step_done;
@@ -784,6 +785,7 @@
     ext->step_frame_fp = 0;
     ext->step_bp_addr = 0;
     ext->step_inlined = 0;
+    ext->step_chk_inline_frame = 0;
     ext->step_continue_mode = RM_RESUME;
     ext->step_mode = RM_RESUME;
 }
@@ -1563,7 +1565,6 @@
         case RM_SKIP_PROLOGUE:
             if (ext->step_cnt == 0) {
                 StackFrame * info = NULL;
-                ext->step_frame_fp = 0;
                 if (get_frame_info(ctx, STACK_TOP_FRAME, &info) < 0) return -1;
                 ext->step_inlined = info->inlined;
                 ext->step_frame_fp = info->fp;
@@ -1776,9 +1777,14 @@
     case RM_REVERSE_STEP_INTO_LINE:
     case RM_REVERSE_STEP_OVER_LINE:
         if (ext->step_cnt == 0) {
-            if (ext->step_code_area == NULL) {
-                if (address_to_line(ctx, addr, addr + 1, update_step_machine_code_area, ext) < 0) return -1;
+            if (ext->step_mode == RM_STEP_INTO_LINE || ext->step_mode == RM_REVERSE_STEP_INTO_LINE) {
+                StackFrame * info = NULL;
+                if (get_frame_info(ctx, STACK_TOP_FRAME, &info) < 0) return -1;
+                ext->step_chk_inline_frame = info->inlined || info->area;
+                ext->step_inlined = info->inlined;
+                ext->step_frame_fp = info->fp;
             }
+            if (address_to_line(ctx, addr, addr + 1, update_step_machine_code_area, ext) < 0) return -1;
             if (ext->step_code_area != NULL) {
                 ext->step_range_start = ext->step_code_area->start_address;
                 ext->step_range_end = ext->step_code_area->end_address;
@@ -1863,6 +1869,15 @@
                 ext->step_range_start += 1;
             }
         }
+        else if (ext->step_chk_inline_frame) {
+            StackFrame * info = NULL;
+            if (get_frame_info(ctx, STACK_TOP_FRAME, &info) < 0) return -1;
+            if (ext->step_inlined != info->inlined || ext->step_frame_fp != info->fp) {
+                ctx->pending_intercept = 1;
+                ext->step_done = REASON_STEP;
+                return 0;
+            }
+        }
         break;
 #if EN_STEP_OVER
     case RM_SKIP_PROLOGUE:
@@ -2161,7 +2176,7 @@
             int done = 0;
             if (step_into_inlined_frame(ctx, &done) < 0) error = errno;
             if (!error && done) {
-                ctx->pending_intercept = 1;
+                ext->step_cnt++;
                 if (run_safe_events_posted < 4) {
                     run_safe_events_posted++;
                     post_event(run_safe_events, NULL);