[heap] Move to page lookups for SemiSpace, NewSpace, and Heap containment methods

src/arm/simulator-arm.cc

Index: src/arm/simulator-arm.cc
diff --git a/src/arm/simulator-arm.cc b/src/arm/simulator-arm.cc
index 06c1bb9724e23ded9b413e6ff11b45b0b1d12ad4..563912bf6fa14e5c0d1dfb06fa033c097f288023 100644
--- a/src/arm/simulator-arm.cc
+++ b/src/arm/simulator-arm.cc
@@ -392,7 +392,8 @@ void ArmDebugger::Debug() {
           HeapObject* obj = reinterpret_cast<HeapObject*>(*cur);
           int value = *cur;
           Heap* current_heap = sim_->isolate_->heap();
-          if (((value & 1) == 0) || current_heap->Contains(obj)) {
+          if (((value & 1) == 0) ||
+              current_heap->ContainsSlow(obj->address())) {

[Hannes Payer (out of office), 2016/02/09 14:40:22]

Why ContainsSlow here? This is not a verifier method.

[Michael Lippautz, 2016/02/09 14:50:28]

As discussed offline: There's a reinterpret_cast above (line: 392) and we are
iterating over raw memory, which could contain anything.

             PrintF(" (");
             if ((value & 1) == 0) {
               PrintF("smi %d", value / 2);