Revision: 17382
Author: [email protected]
Date: Thu Oct 24 19:01:30 2013 UTC
Log: Enable double alignment for constructed arrays.
2nd (of 3) CL's to ensure complete alignment of FixedDoubleArrays.
TEST=
BUG=
[email protected]
Review URL: https://codereview.chromium.org/35133002
Patch from Dusan Milosavljevic <[email protected]>.
http://code.google.com/p/v8/source/detail?r=17382
Modified:
/branches/bleeding_edge/src/hydrogen.cc
=======================================
--- /branches/bleeding_edge/src/hydrogen.cc Wed Oct 23 15:15:15 2013 UTC
+++ /branches/bleeding_edge/src/hydrogen.cc Thu Oct 24 19:01:30 2013 UTC
@@ -2125,6 +2125,11 @@
// Allocate (dealing with failure appropriately)
HAllocate* new_object = builder()->Add<HAllocate>(size_in_bytes,
HType::JSArray(), NOT_TENURED, JS_ARRAY_TYPE);
+
+ // Folded array allocation should be aligned if it has fast double
elements.
+ if (IsFastDoubleElementsKind(kind_)) {
+ new_object->MakeDoubleAligned();
+ }
// Fill in the fields: map, properties, length
HValue* map;
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
