Reviewers: Michael Starzinger,

Description:
Fix compile errors on 64 bit platforms.

Please review this at https://chromiumcodereview.appspot.com/10053009/

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/heap.h
  M     src/heap.cc
  M     test/cctest/test-api.cc


Index: src/heap.cc
===================================================================
--- src/heap.cc (revision 11259)
+++ src/heap.cc (working copy)
@@ -5676,7 +5676,7 @@
 }


-int Heap::PromotedExternalMemorySize() {
+intptr_t Heap::PromotedExternalMemorySize() {
   if (amount_of_external_allocated_memory_
       <= amount_of_external_allocated_memory_at_last_global_gc_) return 0;
   return amount_of_external_allocated_memory_
Index: src/heap.h
===================================================================
--- src/heap.h  (revision 11265)
+++ src/heap.h  (working copy)
@@ -1326,7 +1326,8 @@

   // Adjusts the amount of registered external memory.
   // Returns the adjusted value.
-  inline int AdjustAmountOfExternalAllocatedMemory(int change_in_bytes);
+  inline intptr_t AdjustAmountOfExternalAllocatedMemory(
+      intptr_t change_in_bytes);

   // Allocate uninitialized fixed array.
   MUST_USE_RESULT MaybeObject* AllocateRawFixedArray(int length);
@@ -1646,7 +1647,7 @@
   int gc_post_processing_depth_;

   // Returns the amount of external memory registered since last global gc.
-  int PromotedExternalMemorySize();
+  intptr_t PromotedExternalMemorySize();

   int ms_count_;  // how many mark-sweep collections happened
   unsigned int gc_count_;  // how many gc happened
@@ -1714,7 +1715,7 @@
   intptr_t amount_of_external_allocated_memory_;

   // Caches the amount of external memory registered at the last global gc.
-  int amount_of_external_allocated_memory_at_last_global_gc_;
+  intptr_t amount_of_external_allocated_memory_at_last_global_gc_;

// Indicates that an allocation has failed in the old generation since the
   // last GC.
Index: test/cctest/test-api.cc
===================================================================
--- test/cctest/test-api.cc     (revision 11259)
+++ test/cctest/test-api.cc     (working copy)
@@ -10869,9 +10869,9 @@
   v8::HandleScope outer;
   v8::Persistent<Context> env(Context::New());
   CHECK(!env.IsEmpty());
-  const int kSize = 1024*1024;
+  const intptr_t kSize = 1024*1024;
   CHECK_EQ(v8::V8::AdjustAmountOfExternalAllocatedMemory(kSize), kSize);
-  CHECK_EQ(v8::V8::AdjustAmountOfExternalAllocatedMemory(-kSize), 0);
+  CHECK_EQ(v8::V8::AdjustAmountOfExternalAllocatedMemory(-kSize), 0l);
 }




--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to