[debugger] PrepareFunctionForBreakPoints handles both fcg code and bytecode.

src/objects-inl.h

Index: src/objects-inl.h
diff --git a/src/objects-inl.h b/src/objects-inl.h
index bc21f8eef5821477a0a6eb601642ace7decfeab9..7e4ba3ab098a99f4b8405400f068adb4add5f6b1 100644
--- a/src/objects-inl.h
+++ b/src/objects-inl.h
@@ -6172,8 +6172,8 @@ DebugInfo* SharedFunctionInfo::GetDebugInfo() {
 
 
 bool SharedFunctionInfo::HasDebugCode() {
-  return HasBytecodeArray() ||
-         (HasBaselineCode() && code()->has_debug_break_slots());
+  if (HasBaselineCode()) return code()->has_debug_break_slots();
+  return HasBytecodeArray();

[Michael Starzinger, 2016/08/12 08:27:23]

Just for my own confirmation, this should be the same as the following
predicate, correct?

return (HasBaselineCode() && code()->has_debug_break_slots()) ||
       (!HasBaselineCode() && HasBytecodeArray());

I am fine expressing it either way. Just wanted to confirm my own intuition of
things.

[Yang, 2016/08/12 08:31:27]

Yes. What it says is: if we have fcg code, it needs to have been recompiled for
debugging; if we don't, we need to have bytecode to work with.

 }