Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(77)

Unified Diff: src/compiler/access-info.cc

Issue 2634953002: [turbofan] Don't merge PropertyAccessInfos with different field maps. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | test/mjsunit/regress/regress-crbug-679378.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler/access-info.cc
diff --git a/src/compiler/access-info.cc b/src/compiler/access-info.cc
index 35f7bbc8f4adb1326baca99df40f9e0230891574..f23154aa45aa53674d53f7fb13048347a1adf5ce 100644
--- a/src/compiler/access-info.cc
+++ b/src/compiler/access-info.cc
@@ -150,6 +150,7 @@ bool PropertyAccessInfo::Merge(PropertyAccessInfo const* that) {
// Check if we actually access the same field.
if (this->transition_map_.address() == that->transition_map_.address() &&
this->field_index_ == that->field_index_ &&
+ this->field_map_.address() == that->field_map_.address() &&
this->field_type_->Is(that->field_type_) &&
that->field_type_->Is(this->field_type_) &&
this->field_representation_ == that->field_representation_) {
« no previous file with comments | « no previous file | test/mjsunit/regress/regress-crbug-679378.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698