Tell MSAN that profiler doesn't expect sp to point at initialized memory

Tell MSAN that profiler doesn't expect sp to point at initialized memory

BUG=v8:4444, v8:4151
LOG=N

Committed: https://crrev.com/4fd8c327de7759aa987869a8933ef5a09a670b34
Cr-Commit-Position: refs/heads/master@{#30996}

[yurys, 2015-09-28 19:07:23]

yurys@chromium.org changed reviewers:
+ alph@chromium.org

[yurys, 2015-09-28 19:10:58]

Patchset #1 (id:1) has been deleted

[yurys, 2015-09-28 23:56:06]

Patchset #1 (id:20001) has been deleted

[yurys, 2015-09-28 23:59:23]

Patchset #1 (id:40001) has been deleted

[yurys, 2015-09-29 00:01:47]

The CQ bit was checked by yurys@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-09-29 00:02:03]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1371253002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1371253002/60001

[alph, 2015-09-29 00:04:31]

https://codereview.chromium.org/1371253002/diff/60001/src/profiler/profile-ge...
File src/profiler/profile-generator.cc (right):

https://codereview.chromium.org/1371253002/diff/60001/src/profiler/profile-ge...
src/profiler/profile-generator.cc:610: pc_entry =
code_map_.FindEntry(sample.tos);
hmm, isn't the tos contains a copy of what was at SP when the sample was taken.
Shouldn't you mark the *SP instead in the sampler?

[yurys, 2015-09-29 00:15:30]

https://codereview.chromium.org/1371253002/diff/60001/src/profiler/profile-ge...
File src/profiler/profile-generator.cc (right):

https://codereview.chromium.org/1371253002/diff/60001/src/profiler/profile-ge...
src/profiler/profile-generator.cc:610: pc_entry =
code_map_.FindEntry(sample.tos);
On 2015/09/29 00:04:31, alph wrote:
> hmm, isn't the tos contains a copy of what was at SP when the sample was
taken.
> Shouldn't you mark the *SP instead in the sampler?

Good point. Done.

[yurys, 2015-09-29 00:15:35]

The CQ bit was checked by yurys@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-09-29 00:15:43]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1371253002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1371253002/80001

[yurys, 2015-09-29 00:18:56]

yurys@chromium.org changed reviewers:
+ jkummerow@chromium.org

[commit-bot: I haz the power, 2015-09-29 00:39:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-29 00:39:04]

Dry run: This issue passed the CQ dry run.

[alph, 2015-09-29 01:37:42]

lgtm

[yurys, 2015-09-29 01:57:52]

Jakob can you do OWNERS review?

[Jakob Kummerow, 2015-09-29 07:59:47]

lgtm

[yurys, 2015-09-29 08:17:06]

The CQ bit was checked by yurys@chromium.org

[commit-bot: I haz the power, 2015-09-29 08:17:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1371253002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1371253002/80001

[commit-bot: I haz the power, 2015-09-29 08:19:02]

Committed patchset #2 (id:80001)

[commit-bot: I haz the power, 2015-09-29 08:19:10]

Patchset 2 (id:??) landed as
https://crrev.com/4fd8c327de7759aa987869a8933ef5a09a670b34
Cr-Commit-Position: refs/heads/master@{#30996}