Reviewers: Erik Corry, Vyacheslav Egorov (Google),

Message:
PTAL.


https://chromiumcodereview.appspot.com/10443052/diff/1/src/ia32/stub-cache-ia32.cc
File src/ia32/stub-cache-ia32.cc (left):

https://chromiumcodereview.appspot.com/10443052/diff/1/src/ia32/stub-cache-ia32.cc#oldcode787
src/ia32/stub-cache-ia32.cc:787: __ mov(FieldOperand(receiver_reg,
HeapObject::kMapOffset),
On 2012/05/28 10:15:27, Vyacheslav Egorov (Google) wrote:
All architectures + LStoreNamedField on architectures have the same
problem.

Done.

Description:
Fix missing write barrier in store field stub.

[email protected]
BUG=v8:2143,v8:1465,chromium:129355
TEST=cctest/test-heap/Regress2143


Please review this at https://chromiumcodereview.appspot.com/10443052/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/arm/lithium-arm.h
  M src/arm/lithium-arm.cc
  M src/arm/lithium-codegen-arm.cc
  M src/arm/stub-cache-arm.cc
  M src/hydrogen-instructions.h
  M src/ia32/lithium-codegen-ia32.cc
  M src/ia32/lithium-ia32.h
  M src/ia32/lithium-ia32.cc
  M src/ia32/stub-cache-ia32.cc
  M src/mips/lithium-codegen-mips.cc
  M src/mips/lithium-mips.h
  M src/mips/lithium-mips.cc
  M src/mips/stub-cache-mips.cc
  M src/x64/lithium-codegen-x64.cc
  M src/x64/lithium-x64.h
  M src/x64/lithium-x64.cc
  M src/x64/stub-cache-x64.cc
  M test/cctest/test-heap.cc


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to