Reviewers: Michael Starzinger,

Message:
This patch removes the unused JITCodeEvent::CODE_REMOVED event from the API, and
removes the CodeDeletedEvent from the internal log API.

Mostly there is no effect, but as bug 3918 notes, CODE_REMOVED was indeed
getting fired, but only for code objects in LO space, as far as I can tell
anyway.

Description:
Remove JITCodeEvent::CODE_REMOVED

Instead, JitCodeEventHandlers should remember which ranges they have
added, and evict stale entries in the CODE_ADDED or CODE_MOVED event
handlers.

[email protected]
BUG=v8:3918

Please review this at https://codereview.chromium.org/960493003/

Base URL: https://chromium.googlesource.com/v8/v8@master

Affected files (+6, -101 lines):
  M include/v8.h
  M src/cpu-profiler.h
  M src/cpu-profiler.cc
  M src/gdb-jit.cc
  M src/heap/mark-compact.h
  M src/heap/mark-compact.cc
  M src/heap/spaces.cc
  M src/log.h
  M src/log.cc
  M src/perf-jit.h
  M src/perf-jit.cc
  M src/serialize.cc
  M src/third_party/vtune/vtune-jit.cc
  M test/cctest/test-api.cc
  M test/cctest/test-sampler-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/d/optout.

Reply via email to