CpuProfiler: eliminate cpu-profiler dependency from heap-inl.h
authorloislo <loislo@chromium.org>
Fri, 20 Feb 2015 13:28:42 +0000 (05:28 -0800)
committerCommit bot <commit-bot@chromium.org>
Fri, 20 Feb 2015 13:28:58 +0000 (13:28 +0000)
commit8ba89cce6daf5be4427ba184ec5f2c25ec2a15a4
tree39670d3ed764b7dd85d15183fdf5f60444d79ede
parent9d4c20208d239555b70d755ba3d7a526318437f4
CpuProfiler: eliminate cpu-profiler dependency from heap-inl.h

We accessed to cpu_profiler for tracking SharedFunctionInfo objects movements and used their addresses for generating function_id. Actually we could replace the manually generated shared_id by the pair script_id + position. In this case we can drop SharedFunctionInfo events support from cpu_profiler and remove the dependency.

BTW GetCallUid was used as an unique identifier of the function on the front-end side. Actually it is a hash which might not be unique. So I renamed GetCallUid with GetHash and implemented GetFunctionId method.

BUG=452067
LOG=n

Review URL: https://codereview.chromium.org/941973002

Cr-Commit-Position: refs/heads/master@{#26775}
22 files changed:
src/api.cc
src/arm/lithium-codegen-arm.cc
src/arm64/lithium-codegen-arm64.cc
src/cpu-profiler-inl.h
src/cpu-profiler.cc
src/cpu-profiler.h
src/deoptimizer.cc
src/heap/heap-inl.h
src/ia32/lithium-codegen-ia32.cc
src/ic/handler-compiler.cc
src/ic/ic-compiler.cc
src/log.cc
src/mips/lithium-codegen-mips.cc
src/mips64/lithium-codegen-mips64.cc
src/ppc/lithium-codegen-ppc.cc
src/profile-generator-inl.h
src/profile-generator.cc
src/profile-generator.h
src/runtime/runtime-function.cc
src/serialize.cc
src/x64/lithium-codegen-x64.cc
src/x87/lithium-codegen-x87.cc