Fix MSAN error on arm64 bot.

Fix MSAN error on arm64 bot.

The main fix is to mark stack memory the SafeStackFrameIterator
accesses as initialied.

Drive-by: Make sure we bail out when the simulator is in the
process of updating FP/SP registers.

BUG=v8:5156
Committed: https://crrev.com/ca1dcc9c03f6dc2397c1368aadd59d0aa97a3eff
Cr-Commit-Position: refs/heads/master@{#37315}

[alph, 2016-06-28 01:44:18]

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

[commit-bot: I haz the power, 2016-06-28 01:44:21]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-28 02:08:23]

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

[commit-bot: I haz the power, 2016-06-28 02:08:24]

Dry run: This issue passed the CQ dry run.

[alph, 2016-06-28 06:10:53]

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

[Yang, 2016-06-28 06:20:18]

LGTM. Thanks!

https://codereview.chromium.org/2104763002/diff/1/src/frames.cc
File src/frames.cc (right):

https://codereview.chromium.org/2104763002/diff/1/src/frames.cc#newcode307
src/frames.cc:307: #if defined(USE_SIMULATOR)
I think we can use the MSAN macro without this ifdef. If MSAN is not used (which
is the case for non-simulator builds for arm64), the MSAN macro is replaced by
nothing.

[alph, 2016-06-28 07:27:17]

https://codereview.chromium.org/2104763002/diff/1/src/frames.cc
File src/frames.cc (right):

https://codereview.chromium.org/2104763002/diff/1/src/frames.cc#newcode307
src/frames.cc:307: #if defined(USE_SIMULATOR)
On 2016/06/28 06:20:18, Yang wrote:
> I think we can use the MSAN macro without this ifdef. If MSAN is not used
(which
> is the case for non-simulator builds for arm64), the MSAN macro is replaced by
> nothing.

Done.

[alph, 2016-06-28 07:27:45]

The CQ bit was checked by alph@chromium.org

[alph, 2016-06-28 07:27:46]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/2104763002/#ps20001
(title: "Remove ifdef around MSAN macro")

[commit-bot: I haz the power, 2016-06-28 07:27:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-28 07:57:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-28 07:58:09]

Description was changed from

==========
Fix MSAN error on arm64 bot.

The main fix is to mark stack memory the SafeStackFrameIterator
accesses as initialied.

Drive-by: Make sure we bail out when the simulator is in the
process of updating FP/SP registers.

BUG=v8:5156
==========

to

==========
Fix MSAN error on arm64 bot.

The main fix is to mark stack memory the SafeStackFrameIterator
accesses as initialied.

Drive-by: Make sure we bail out when the simulator is in the
process of updating FP/SP registers.

BUG=v8:5156

Committed: https://crrev.com/ca1dcc9c03f6dc2397c1368aadd59d0aa97a3eff
Cr-Commit-Position: refs/heads/master@{#37315}
==========

[commit-bot: I haz the power, 2016-06-28 07:58:10]

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