Chromium Code Reviews

Unified Diff: src/x87/full-codegen-x87.cc

Issue 914773002: Fix issue with super and computed property names (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
« no previous file with comments | « src/x64/full-codegen-x64.cc ('k') | test/mjsunit/harmony/class-computed-property-names-super.js » ('j') | test/mjsunit/mjsunit.status » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/x87/full-codegen-x87.cc
diff --git a/src/x87/full-codegen-x87.cc b/src/x87/full-codegen-x87.cc
index a1c2960b4d70637de8dfaf685d1af06f0e0f281d..f54b2bee84c1b9263529dab6491f7406f386223e 100644
--- a/src/x87/full-codegen-x87.cc
+++ b/src/x87/full-codegen-x87.cc
@@ -1727,6 +1727,7 @@ void FullCodeGenerator::VisitObjectLiteral(ObjectLiteral* expr) {
} else {
EmitPropertyKey(property, expr->GetIdForProperty(property_index));
VisitForStackValue(value);
+ EmitSetHomeObjectIfNeeded(value, 2);
switch (property->kind()) {
case ObjectLiteral::Property::CONSTANT:
« no previous file with comments | « src/x64/full-codegen-x64.cc ('k') | test/mjsunit/harmony/class-computed-property-names-super.js » ('j') | test/mjsunit/mjsunit.status » ('J')

Powered by Google App Engine