Reviewers: danno,
Message:
I updated the code according to the comments. And I updated one test case to
cover the new added event type.
please help to review it.
Description:
This patch is the propagation version of
https://codereview.chromium.org/10824032 patch
which is for "Enables V8 integration with Intel Vtune performance analysis
tool".
we divided that implementation into 2 parts. This is the first part. we
enahance
the
current JitCodeEventHandler implementation according to Danno and Michael
Starzinger
's suggestion.
1, we added the CODE_ADD_LINE_POS_INFO, CODE_START_LINE_INFO_RECORDING,
CODE_END_LINE_INFO_RECORDING event and the corresponding functionality.
2, Add the JITCodeLineInfo struct to record the code line info. I add this
definition
because Danno mentioned that "we'd like to cleanup and decouple the
external
debugging functionality"
3, some other small changes.
Please review this at https://codereview.chromium.org/11552033/
SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/
Affected files:
M include/v8.h
M src/assembler.h
M src/assembler.cc
M src/full-codegen.cc
M src/lithium.cc
M src/log.h
M src/log.cc
M test/cctest/test-api.cc
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
