tree 0329edb35ee28fc0331e41829d853eb0809ec738
parent c70666a3a3a5e0f63675a0bc3a383bfddc14e408
author Alexander Markov <alexmarkov@google.com> 1539972094 +0000
committer commit-bot@chromium.org <commit-bot@chromium.org> 1539972094 +0000

[vm/interpreter] Fix interpreter frame recognition in stack walking, v2

When GC is visiting pointers, it scans mutator thread even if it is not
active. It could happen if a thread has (temporarily) exitted isolate.
In such case, stack frame iteration would not find interpreter and
would not recognize interpreter frames. An attempt to visit interpreter
frame as non-interpreter causes crashes/assertions.

The fix is to move interpreter from Isolate to Thread. This way
interpreter is available for stack walking even if thread doesn't have
isolate.

This is the re-land of https://dart-review.googlesource.com/c/sdk/+/80760
with the fix for precompiler failures.

Change-Id: I1fa054d43b86aa49c48490d0951b876b7c7218d8
Reviewed-on: https://dart-review.googlesource.com/c/80782
Auto-Submit: Alexander Markov <alexmarkov@google.com>
Reviewed-by: Régis Crelier <regis@google.com>
Commit-Queue: Régis Crelier <regis@google.com>