Reviewers: Yang,
Description:
Version 3.24.35.28 (merged r20127)
Fix for performance issue 353661 - 23% regression in dromaeo.
BUG=353661
LOG=N
R=yang...@chromium.org
Please review this at https://codereview.chromium.org/225403004/
SVN Base: https://v8.googlecode.com/svn/branches/3.24
Affected files (+3, -2 lines):
M src/ic.cc
M src/version.cc
Index: src/ic.cc
diff --git a/src/ic.cc b/src/ic.cc
index
71f821febd0ff8a4f659b7fbbe614fb0a34b42a7..aed06f923f7c7b45ad614cead96f708490796c17
100644
--- a/src/ic.cc
+++ b/src/ic.cc
@@ -1438,7 +1438,8 @@ Handle<Code>
KeyedStoreIC::StoreElementStub(Handle<JSObject> receiver,
if (IsTransitionStoreMode(store_mode)) {
transitioned_receiver_map = ComputeTransitionedMap(receiver,
store_mode);
}
- if (receiver_map.is_identical_to(previous_receiver_map) ||
+ if ((receiver_map.is_identical_to(previous_receiver_map) &&
+ IsTransitionStoreMode(store_mode)) ||
IsTransitionOfMonomorphicTarget(
MapToType<HeapType>(transitioned_receiver_map, isolate()))) {
// If the "old" and "new" maps are in the same elements map family,
or
Index: src/version.cc
diff --git a/src/version.cc b/src/version.cc
index
5ae3b854dd8f6a1b3bb3c79dfe0a56b76f8f6638..304388f66e9e14ab463f2eb767e44287be79ee5b
100644
--- a/src/version.cc
+++ b/src/version.cc
@@ -35,7 +35,7 @@
#define MAJOR_VERSION 3
#define MINOR_VERSION 24
#define BUILD_NUMBER 35
-#define PATCH_LEVEL 27
+#define PATCH_LEVEL 28
// Use 1 for candidates and 0 otherwise.
// (Boolean macro values are not supported by all preprocessors.)
#define IS_CANDIDATE_VERSION 0
--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
