Revision: 11638
Author:   [email protected]
Date:     Wed May 23 14:15:14 2012
Log:      Fix build and test failures from packed array optimizations.

[email protected]

Review URL: https://chromiumcodereview.appspot.com/10332317
http://code.google.com/p/v8/source/detail?r=11638

Modified:
 /branches/bleeding_edge/src/x64/macro-assembler-x64.cc
 /branches/bleeding_edge/test/mjsunit/unbox-double-arrays.js

=======================================
--- /branches/bleeding_edge/src/x64/macro-assembler-x64.cc Wed May 23 07:24:29 2012 +++ /branches/bleeding_edge/src/x64/macro-assembler-x64.cc Wed May 23 14:15:14 2012
@@ -4059,7 +4059,7 @@
   movq(scratch, Operand(scratch,
Context::SlotOffset(Context::JS_ARRAY_MAPS_INDEX)));

-  size_t offset = expected_kind * kPointerSize +
+  int offset = expected_kind * kPointerSize +
       FixedArrayBase::kHeaderSize;
   cmpq(map_in_out, FieldOperand(scratch, offset));
   j(not_equal, no_map_match);
=======================================
--- /branches/bleeding_edge/test/mjsunit/unbox-double-arrays.js Wed May 23 07:24:29 2012 +++ /branches/bleeding_edge/test/mjsunit/unbox-double-arrays.js Wed May 23 14:15:14 2012
@@ -278,7 +278,8 @@
                       expected_array_value(7));

   %DeoptimizeFunction(test_various_loads6);
-  gc();
+  %ClearFunctionTypeFeedback(test_various_stores);
+  %ClearFunctionTypeFeedback(test_various_loads7);

   // Test stores for non-NaN.
   var large_array = new allocator(large_array_size);

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

Reply via email to