Status: Accepted
Owner: [email protected]
CC: [email protected],  [email protected]
Labels: Type-Bug Priority-Medium

New issue 3170 by [email protected]: A64: Enable AssertStackConsistency in PrepareForPush.
http://code.google.com/p/v8/issues/detail?id=3170

AssertStackConsistency was added to PrepareForPush in r19474, but disabled in r19491 because it generates too much code and causes some tests to fail (intermittently). We should find a way to enable this assertion, either by optimizing the size of some generated code, or by increasing the size of the relevant buffers.

This assertion is known to trigger the following failures on a debug build with snapshot=off:

# Fatal error in ../src/a64/assembler-a64.cc, line 2342
# external code buffer is too small
  - cctest/test-api/MultipleIsolatesOnIndividualThreads
  - cctest/test-lockers/MultithreadedParallelIsolates
  - cctest/test-lockers/LockAndUnlockDifferentIsolates
  - cctest/test-lockers/ExtensionsRegistration

# Fatal error in ../src/spaces.cc, line 2378
# CHECK(size_in_bytes <= kMaxBlockSize) failed
  - cctest/test-serialize/DeserializeFromSecondSerialization
  - cctest/test-serialize/DeserializeFromSecondSerializationAndRunScript2

# Fatal error in ../test/cctest/test-spaces.cc, line 412
# CHECK_EQ(1, isolate->heap()->paged_space(i)->CountTotalPages()) failed
#   Expected: 1
#   Found: 2
  - cctest/test-spaces/SizeOfFirstPageIsLargeEnough


--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to