Reviewers: danno,

Message:
hello. Daniel. The original CL
https://codereview.chromium.org/11552033
can not be accessed. I don't know the reason. So I create
this CL. Please help to review.

As far as the "code move" support. My plan is that currently we use the
"--nocompact_code_space" flag to disable it. I will submit another patch to
support it so that the current patch can be accepted in time.
Is it ok?


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/12223027/

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     include/v8.h
  M     src/api.cc
  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
--- 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.


Reply via email to