commit 0bfcbdd4726920755e51dab28c18ab93e050819b
author Peter Marshall <petermarshall@chromium.org> Thu May 24 11:09:16 2018
committer Commit Bot <commit-bot@chromium.org> Thu May 24 11:55:27 2018
tree f45c6ee29ddd5a1b496afb473f91a82b56c59b9a
parent 170418b2124c44bc16cb485a909f0d2a040e1009

[cpu-profiler] Only store deopt inline frames for functions that need it

We store deopt inline frames for all functions when we receive the code
creation event. We only ever use this information for code which is
deoptimized. Given that we receive code deopt events, we can just store
this information when the code is deoptimized.

At the time of the code deopt event, we also know the associated
deopt_id. That means we don't need to store a map of deopt_ids to
vectors of frames, because we will only ever access the frames for the
deopt_id that is already set.

This means we store way less data, particularly for long-running
processes which see fewer deopts. This saves 10MiB peak memory on the
node server example.

Bug: v8:7719
Change-Id: If6cf5ec413848e4c9f3c1e2106366ae2adae6fb1
Reviewed-on: https://chromium-review.googlesource.com/1050289
Commit-Queue: Peter Marshall <petermarshall@chromium.org>
Reviewed-by: Alexei Filippov <alph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#53330}