Fixed compound/count operations with getter-only accessor properties.
authorsvenpanne@chromium.org <svenpanne@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Fri, 3 Aug 2012 09:45:08 +0000 (09:45 +0000)
committersvenpanne@chromium.org <svenpanne@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Fri, 3 Aug 2012 09:45:08 +0000 (09:45 +0000)
commit83fc4205f69cd59503d16baf7c4e59349d05dd0a
tree3cc223dcc6276f4799204c90a97ce605ac4ec4e2
parent680c4a7a134762096e6ad7848bd93be0b590dacd
Fixed compound/count operations with getter-only accessor properties.

The underlying problem is that for compound/count operations we use the *load*
type feedback for storing, too. For normal properties this doesn't matter, but
for accessor properties we should better use the *store* type feedback, which
would be available, too. This consistent feedback usage could be guaranteed if
we removed the heavy copy-n-paste in the crankshaft code generation for
compound/count operations and assignments/property loads.

To be on the safe side, we postpone this refactoring and do a quick and easily
mergeable fix.

BUG=140083
TEST=mjsunit/regress/regress-crbug-140083.js

Review URL: https://chromiumcodereview.appspot.com/10828146

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12252 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
src/hydrogen.cc
test/mjsunit/regress/regress-crbug-140083.js [new file with mode: 0644]