From 5ca3cec023d7c4f0803633b869a3fce7f8ad38e5 Mon Sep 17 00:00:00 2001 From: "yangguo@chromium.org" Date: Tue, 10 Jun 2014 09:16:29 +0000 Subject: [PATCH] Reuse RelocInfo's IsPatchedDebugBreakSlotSequence() implementation. RelocInfo::IsPatchedDebugBreakSlotSequence() already does this assembly check, so just reuse it in BreakLocationIterator::IsDebugBreakAtSlot(), like the other debug ports do. R=yangguo@chromium.org Review URL: https://codereview.chromium.org/321743002 Patch from Thiago Farina . git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@21734 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- src/x64/debug-x64.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/x64/debug-x64.cc b/src/x64/debug-x64.cc index bdb9e51..4703e42 100644 --- a/src/x64/debug-x64.cc +++ b/src/x64/debug-x64.cc @@ -48,7 +48,7 @@ bool Debug::IsDebugBreakAtReturn(v8::internal::RelocInfo* rinfo) { bool BreakLocationIterator::IsDebugBreakAtSlot() { ASSERT(IsDebugBreakSlot()); // Check whether the debug break slot instructions have been patched. - return !Assembler::IsNop(rinfo()->pc()); + return rinfo()->IsPatchedDebugBreakSlotSequence(); } -- 2.7.4