[heap] make inline-allocation-observers precise

src/heap/spaces.h

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_HEAP_SPACES_H_
 #define V8_HEAP_SPACES_H_
 
 #include "src/allocation.h"
 #include "src/atomic-utils.h"
 #include "src/base/atomicops.h"
@@ skipping 1475 matching lines @@
     return top_;
   }
 
   Address* top_address() { return &top_; }
 
   INLINE(void set_limit(Address limit)) {
     limit_ = limit;
   }
 
   INLINE(Address limit()) const {
-    SLOW_DCHECK(limit_ == NULL ||
-                (reinterpret_cast<intptr_t>(limit_) & kHeapObjectTagMask) ==
-                    0);
     return limit_;
   }
 
   Address* limit_address() { return &limit_; }
 
 #ifdef DEBUG
   bool VerifyPagedAllocation() {
     return (Page::FromAllocationTop(top_) == Page::FromAllocationTop(limit_)) &&
            (top_ <= limit_);
   }
@@ skipping 997 matching lines @@
   // Last page returned.
   NewSpacePage* last_page_;
 };
 
 // -----------------------------------------------------------------------------
 // Allows observation of inline allocation in the new space.
 class InlineAllocationObserver {
  public:
   explicit InlineAllocationObserver(intptr_t step_size)
       : step_size_(step_size), bytes_to_next_step_(step_size) {
-    DCHECK(step_size >= kPointerSize && (step_size & kHeapObjectTagMask) == 0);
+    DCHECK(step_size >= kPointerSize);
   }
   virtual ~InlineAllocationObserver() {}
 
  private:
   intptr_t step_size() const { return step_size_; }
   intptr_t bytes_to_next_step() const { return bytes_to_next_step_; }
 
-  // Pure virtual method provided by the subclasses that gets called when more
-  // than step_size byte have been allocated.
+  // Pure virtual method provided by the subclasses that gets called when at
+  // least step_size bytes have been allocated.
   virtual void Step(int bytes_allocated) = 0;
 
   // Called each time the new space does an inline allocation step. This may be
   // more frequently than the step_size we are monitoring (e.g. when there are
-  // multiple observers, or when page or space boundary is encountered.) The
-  // Step method is only called once more than step_size bytes have been
-  // allocated.
+  // multiple observers, or when page or space boundary is encountered.)
   void InlineAllocationStep(int bytes_allocated) {
     bytes_to_next_step_ -= bytes_allocated;
     if (bytes_to_next_step_ <= 0) {
       Step(static_cast<int>(step_size_ - bytes_to_next_step_));
       bytes_to_next_step_ = step_size_;
     }
   }
 
   intptr_t step_size_;
   intptr_t bytes_to_next_step_;
@@ skipping 581 matching lines @@
     count = 0;
   }
   // Must be small, since an iteration is used for lookup.
   static const int kMaxComments = 64;
 };
 #endif
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_HEAP_SPACES_H_