Always update raw pointers when handling interrupts inside RegExp code.

Always update raw pointers when handling interrupts inside RegExp code.

R=mstarzinger@chromium.org
BUG=chromium:469480
LOG=N

Committed: https://crrev.com/c67cb287a901ddf03d4ae4dafcf431d09fd3e22c
Cr-Commit-Position: refs/heads/master@{#27615}

[Yang, 2015-03-26 15:14:13]

On 2015/03/26 15:13:45, Yang wrote:

This is only the ia32 port of this patch. I'd like a preliminary review before I
start porting.

[Michael Starzinger, 2015-03-26 15:26:04]

Approach LGTM.

https://codereview.chromium.org/1034173002/diff/1/src/isolate.cc
File src/isolate.cc (right):

https://codereview.chromium.org/1034173002/diff/1/src/isolate.cc#newcode836
src/isolate.cc:836: if (FLAG_verify_heap && FLAG_stress_compaction) {
Needs to also be #ifdef'ed with VERIFY_HEAP here.

[Yang, 2015-03-27 12:37:11]

yangguo@chromium.org changed reviewers:
+ dusan.milosavljevic@imgtec.com, plind44@gmail.com

[Yang, 2015-03-27 12:37:12]

Added ports and addressed comment.

The test fails on MIPS64, but that's due to an unrelated bug on a code path that
has not been tested until now. I filed a bug for this
(https://code.google.com/p/v8/issues/detail?id=3992).

[Yang, 2015-03-30 13:11:48]

On 2015/03/27 12:37:12, Yang wrote:
> Added ports and addressed comment.
> 
> The test fails on MIPS64, but that's due to an unrelated bug on a code path
that
> has not been tested until now. I filed a bug for this
> (https://code.google.com/p/v8/issues/detail?id=3992).

Michael, do you want to take another look or is this good to go?

[Michael Starzinger, 2015-03-30 14:18:49]

Still LGTM.

[Yang, 2015-03-30 15:11:39]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2015-03-30 15:11:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1034173002/20001

[commit-bot: I haz the power, 2015-03-30 15:21:07]

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

[commit-bot: I haz the power, 2015-03-30 15:21:08]

Try jobs failed on following builders:
  v8_win64_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel/builds/3409)

[Yang, 2015-04-07 09:15:50]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2015-04-07 09:15:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1034173002/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2015-04-07 09:16:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1034173002/40001

[commit-bot: I haz the power, 2015-04-07 09:44:56]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-04-07 09:45:12]

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