Revision: 16209
Author: [email protected]
Date: Fri Aug 16 19:52:29 2013 UTC
Log: Remove check for empty handle for CodeGenerator::MakeCodeEpilogue.
[email protected]
Review URL: https://codereview.chromium.org/22840002
http://code.google.com/p/v8/source/detail?r=16209
Modified:
/branches/bleeding_edge/src/codegen.cc
/branches/bleeding_edge/src/full-codegen.cc
/branches/bleeding_edge/src/lithium.cc
=======================================
--- /branches/bleeding_edge/src/codegen.cc Wed Jul 24 14:28:56 2013 UTC
+++ /branches/bleeding_edge/src/codegen.cc Fri Aug 16 19:52:29 2013 UTC
@@ -114,11 +114,9 @@
Handle<Code> code =
isolate->factory()->NewCode(desc, flags, masm->CodeObject(),
false, is_crankshafted);
- if (!code.is_null()) {
- isolate->counters()->total_compiled_code_size()->Increment(
- code->instruction_size());
- code->set_prologue_offset(info->prologue_offset());
- }
+ isolate->counters()->total_compiled_code_size()->Increment(
+ code->instruction_size());
+ code->set_prologue_offset(info->prologue_offset());
return code;
}
=======================================
--- /branches/bleeding_edge/src/full-codegen.cc Mon Aug 5 12:43:04 2013 UTC
+++ /branches/bleeding_edge/src/full-codegen.cc Fri Aug 16 19:52:29 2013 UTC
@@ -350,21 +350,17 @@
code->set_back_edge_table_offset(table_offset);
code->set_back_edges_patched_for_osr(false);
CodeGenerator::PrintCode(code, info);
- info->SetCode(code); // May be an empty handle.
+ info->SetCode(code);
#ifdef ENABLE_GDB_JIT_INTERFACE
- if (FLAG_gdbjit && !code.is_null()) {
+ if (FLAG_gdbjit) {
GDBJITLineInfo* lineinfo =
masm.positions_recorder()->DetachGDBJITLineInfo();
-
GDBJIT(RegisterDetailedLineInfo(*code, lineinfo));
}
#endif
- if (!code.is_null()) {
- void* line_info =
- masm.positions_recorder()->DetachJITHandlerData();
- LOG_CODE_EVENT(isolate, CodeEndLinePosInfoRecordEvent(*code,
line_info));
- }
- return !code.is_null();
+ void* line_info = masm.positions_recorder()->DetachJITHandlerData();
+ LOG_CODE_EVENT(isolate, CodeEndLinePosInfoRecordEvent(*code, line_info));
+ return true;
}
=======================================
--- /branches/bleeding_edge/src/lithium.cc Fri Aug 2 09:53:11 2013 UTC
+++ /branches/bleeding_edge/src/lithium.cc Fri Aug 16 19:52:29 2013 UTC
@@ -461,12 +461,10 @@
CodeGenerator::MakeCodeEpilogue(&assembler, flags, info());
generator.FinishCode(code);
code->set_is_crankshafted(true);
- if (!code.is_null()) {
- void* jit_handler_data =
- assembler.positions_recorder()->DetachJITHandlerData();
- LOG_CODE_EVENT(info()->isolate(),
- CodeEndLinePosInfoRecordEvent(*code,
jit_handler_data));
- }
+ void* jit_handler_data =
+ assembler.positions_recorder()->DetachJITHandlerData();
+ LOG_CODE_EVENT(info()->isolate(),
+ CodeEndLinePosInfoRecordEvent(*code, jit_handler_data));
CodeGenerator::PrintCode(code, info());
return code;
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
