Reviewers: Michael Starzinger, Sven Panne, yurys, alph,
Message:
PTAL
Description:
Propagate DeoptInfo to cpu-profiler
1) Deoptimizer::Reason was replaced with Deoptimizer::DeoptInfo
because it also has raw position. Also the old name clashes with DeoptReason
enum.
2) c_entry_fp assignment call was added to EntryGenerator::Generate
So we can calculate sp and have a chance to record the stack for the
deopting
function.
btw it makes the test stable.
3) new kind of CodeEvents was added to cpu-profiler
4) GetDeoptInfo method was extracted from PrintDeoptLocation.
So it could be reused in cpu profiler.
BUG=452067
LOG=n
Please review this at https://codereview.chromium.org/910773002/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+233, -92 lines):
M src/arm/deoptimizer-arm.cc
M src/arm/lithium-codegen-arm.cc
M src/arm64/deoptimizer-arm64.cc
M src/arm64/lithium-codegen-arm64.cc
M src/cpu-profiler.h
M src/cpu-profiler.cc
M src/cpu-profiler-inl.h
M src/deoptimizer.h
M src/deoptimizer.cc
M src/ia32/deoptimizer-ia32.cc
M src/ia32/lithium-codegen-ia32.cc
M src/lithium-codegen.h
M src/lithium-codegen.cc
M src/log.h
M src/log.cc
M src/mips/deoptimizer-mips.cc
M src/mips/lithium-codegen-mips.cc
M src/mips64/lithium-codegen-mips64.cc
M src/objects.cc
M src/ppc/lithium-codegen-ppc.cc
M src/profile-generator.h
M src/profile-generator-inl.h
M src/x64/deoptimizer-x64.cc
M src/x64/lithium-codegen-x64.cc
M src/x87/lithium-codegen-x87.cc
M test/cctest/test-cpu-profiler.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.
