Fix issue with multiple properties and emit store.
authorarv <arv@chromium.org>
Wed, 4 Feb 2015 20:44:01 +0000 (12:44 -0800)
committerCommit bot <commit-bot@chromium.org>
Wed, 4 Feb 2015 20:44:12 +0000 (20:44 +0000)
commitcd661fdc277a75dec8506f038e3d5904b29de141
tree0d7ae8d5e9374c21166b0fa9f6f46d2dd5ec9dbb
parent59525f07dfa4d80f168b33ee6459336bdc1e4184
Fix issue with multiple properties and emit store.

We used to have a parse error for conflicting property keys. This check
was removed to match ES6 (SpiderMonkey & Chakra already made this change).
Since this check was removed we ended up with a few new cases when
generating code. For example, accessors always generated code even if
those should have been shadowed by a data property.

BUG=v8:3856
LOG=Y
R=adamk, dslomov@chromium.org

Review URL: https://codereview.chromium.org/895693002

Cr-Commit-Position: refs/heads/master@{#26441}
src/arm/full-codegen-arm.cc
src/arm64/full-codegen-arm64.cc
src/ast.cc
src/compiler/ast-graph-builder.cc
src/ia32/full-codegen-ia32.cc
src/mips/full-codegen-mips.cc
src/mips64/full-codegen-mips64.cc
src/ppc/full-codegen-ppc.cc
src/x64/full-codegen-x64.cc
src/x87/full-codegen-x87.cc
test/mjsunit/object-literal-multiple-fields.js [new file with mode: 0644]