Author: olehougaard
Date: Mon Jan 26 05:21:49 2009
New Revision: 1152
Modified:
branches/bleeding_edge/src/serialize.h
branches/bleeding_edge/test/cctest/cctest.status
Log:
Reintroduced a fix for an alignment issue on ARM. I had reverted it by
mistake. This fixes the serialization test suite on ARM.
Review URL: http://codereview.chromium.org/18751
Modified: branches/bleeding_edge/src/serialize.h
==============================================================================
--- branches/bleeding_edge/src/serialize.h (original)
+++ branches/bleeding_edge/src/serialize.h Mon Jan 26 05:21:49 2009
@@ -225,8 +225,8 @@
}
int GetInt() {
- int result = *reinterpret_cast<const int*>(str_);
- str_ += sizeof(result);
+ int result;
+ GetBytes(reinterpret_cast<Address>(&result), sizeof(result));
return result;
}
Modified: branches/bleeding_edge/test/cctest/cctest.status
==============================================================================
--- branches/bleeding_edge/test/cctest/cctest.status (original)
+++ branches/bleeding_edge/test/cctest/cctest.status Mon Jan 26 05:21:49
2009
@@ -33,10 +33,6 @@
[ $arch == arm ]
test-debug: SKIP
-test-serialize/Deserialize: SKIP
-test-serialize/DeserializeAndRunScript: SKIP
-test-serialize/DeserializeNatives: SKIP
-test-serialize/DeserializeExtensions: SKIP
# BUG(113): Test seems flaky on ARM.
test-spaces/LargeObjectSpace: PASS || FAIL
--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---
