Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(724)

Unified Diff: src/lithium.cc

Issue 10824032: Enables V8 integration with the Intel VTune performance analysis tool. This allows the VTune profi… (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/lithium.cc
===================================================================
--- src/lithium.cc (revision 12182)
+++ src/lithium.cc (working copy)
@@ -419,6 +419,10 @@
Handle<Code> LChunk::Codegen() {
MacroAssembler assembler(info()->isolate(), NULL, 0);
+#ifdef ENABLE_VTUNE_JIT_INTERFACE
+ if (VTUNERUNNING)
+ assembler.positions_recorder()->StartVtuneJITLineInfoRecording();
+#endif
LCodeGen generator(this, &assembler, info());
MarkEmptyBlocks();
@@ -432,6 +436,16 @@
Handle<Code> code =
CodeGenerator::MakeCodeEpilogue(&assembler, flags, info());
generator.FinishCode(code);
+#ifdef ENABLE_VTUNE_JIT_INTERFACE
+ if (VTUNERUNNING) {
+ if (!code.is_null()) {
+ VtuneJITLineInfo* vtunelineinfo =
+ assembler.positions_recorder()->DetachVtuneJITLineInfo();
+
+ VTUNEJIT(RegisterDetailedLineInfo(*code, vtunelineinfo));
+ }
+ }
+#endif
CodeGenerator::PrintCode(code, info());
return code;
}

Powered by Google App Engine
This is Rietveld 408576698