| Index: src/profiler/profiler-listener.cc
|
| diff --git a/src/profiler/profiler-listener.cc b/src/profiler/profiler-listener.cc
|
| index 838076120b9f8625944cceccf03a68575926ead7..a25ccf11ac5a1519fad135e971df44b2d08f7f4c 100644
|
| --- a/src/profiler/profiler-listener.cc
|
| +++ b/src/profiler/profiler-listener.cc
|
| @@ -88,51 +88,23 @@ void ProfilerListener::CodeCreateEvent(CodeEventListener::LogEventsAndTags tag,
|
| JITLineInfoTable* line_table = NULL;
|
| if (script) {
|
| line_table = new JITLineInfoTable();
|
| - if (abstract_code->IsCode()) {
|
| - Code* code = abstract_code->GetCode();
|
| - int start_position = shared->start_position();
|
| - int end_position = shared->end_position();
|
| - if (code->kind() == Code::FUNCTION ||
|
| - (code->is_optimized_code() && !code->is_turbofanned())) {
|
| - for (SourcePositionTableIterator it(code->source_position_table());
|
| - !it.done(); it.Advance()) {
|
| - int position = it.source_position();
|
| - // TODO(alph): in case of inlining the position may correspond to an
|
| - // inlined function source code. Do not collect positions that fall
|
| - // beyond the function source code. There's however a chance the
|
| - // inlined function has similar positions but in another script. So
|
| - // the proper fix is to store script_id in some form along with the
|
| - // inlined function positions.
|
| - if (position < start_position || position >= end_position) continue;
|
| - int line_number = script->GetLineNumber(it.source_position()) + 1;
|
| - int pc_offset = it.code_offset() + Code::kHeaderSize;
|
| - line_table->SetPosition(pc_offset, line_number);
|
| - }
|
| - } else {
|
| - for (RelocIterator it(code); !it.done(); it.next()) {
|
| - RelocInfo* reloc_info = it.rinfo();
|
| - if (!RelocInfo::IsPosition(reloc_info->rmode())) continue;
|
| - int position = static_cast<int>(reloc_info->data());
|
| - // TODO(alph): in case of inlining the position may correspond to an
|
| - // inlined function source code. Do not collect positions that fall
|
| - // beyond the function source code. There's however a chance the
|
| - // inlined function has similar positions but in another script. So
|
| - // the proper fix is to store script_id in some form along with the
|
| - // inlined function positions.
|
| - if (position < start_position || position >= end_position) continue;
|
| - int pc_offset = static_cast<int>(reloc_info->pc() - code->address());
|
| - int line_number = script->GetLineNumber(position) + 1;
|
| - line_table->SetPosition(pc_offset, line_number);
|
| - }
|
| - }
|
| - } else {
|
| - BytecodeArray* bytecode = abstract_code->GetBytecodeArray();
|
| - SourcePositionTableIterator it(bytecode->source_position_table());
|
| - for (; !it.done(); it.Advance()) {
|
| - int line_number = script->GetLineNumber(it.source_position()) + 1;
|
| - int pc_offset = it.code_offset() + BytecodeArray::kHeaderSize;
|
| - line_table->SetPosition(pc_offset, line_number);
|
| - }
|
| + int offset = abstract_code->IsCode() ? Code::kHeaderSize
|
| + : BytecodeArray::kHeaderSize;
|
| + int start_position = shared->start_position();
|
| + int end_position = shared->end_position();
|
| + for (SourcePositionTableIterator it(abstract_code->source_position_table());
|
| + !it.done(); it.Advance()) {
|
| + int position = it.source_position();
|
| + // TODO(alph): in case of inlining the position may correspond to an
|
| + // inlined function source code. Do not collect positions that fall
|
| + // beyond the function source code. There's however a chance the
|
| + // inlined function has similar positions but in another script. So
|
| + // the proper fix is to store script_id in some form along with the
|
| + // inlined function positions.
|
| + if (position < start_position || position >= end_position) continue;
|
| + int line_number = script->GetLineNumber(position) + 1;
|
| + int pc_offset = it.code_offset() + offset;
|
| + line_table->SetPosition(pc_offset, line_number);
|
| }
|
| }
|
| rec->entry = NewCodeEntry(
|
|
|