[debugger] PrepareFunctionForBreakPoints handles both fcg code and bytecode.

src/objects-inl.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // Review notes:
 //
 // - The use of macros in these inline functions may seem superfluous
 // but it is absolutely needed to make sure gcc generates optimal
 // code. gcc is not happy when attempting to inline too deep.
 //
@@ skipping 6154 matching lines @@
 }
 
 
 DebugInfo* SharedFunctionInfo::GetDebugInfo() {
   DCHECK(HasDebugInfo());
   return DebugInfo::cast(debug_info());
 }
 
 
 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.

 }
 
 
 bool SharedFunctionInfo::IsApiFunction() {
   return function_data()->IsFunctionTemplateInfo();
 }
 
 
 FunctionTemplateInfo* SharedFunctionInfo::get_api_func_data() {
   DCHECK(IsApiFunction());
@@ skipping 2021 matching lines @@
 #undef WRITE_INT64_FIELD
 #undef READ_BYTE_FIELD
 #undef WRITE_BYTE_FIELD
 #undef NOBARRIER_READ_BYTE_FIELD
 #undef NOBARRIER_WRITE_BYTE_FIELD
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_OBJECTS_INL_H_