AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

Fix potential deopt-loop introduced in r17143.

Browse Source 
TBR=verwaest@chromium.org
BUG=

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

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17144 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
This commit is contained in:
olivf@chromium.org 2013-10-10 14:54:31 +00:00
parent 1384cc0a3d
commit caeb19fa28
1 changed files with 12 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/code-stubs.cc
View File

@ -584,11 +584,18 @@ void BinaryOpStub::UpdateStatus(Handle<Object> left,
op_ == Token::ADD); op_ == Token::ADD);
if (old_state == GetExtraICState()) { if (old_state == GetExtraICState()) {
// Since the fpu is to precise, we might bail out on numbers which // Tagged operations can lead to non-truncating HChanges
// actually would truncate with 64 bit precision. if (left->IsUndefined()) {
ASSERT(!CpuFeatures::IsSupported(SSE2) && left_state_ = GENERIC;
result_state_ <= INT32); } else if (right->IsUndefined()) {
result_state_ = NUMBER; right_state_ = GENERIC;
} else {
// Since the fpu is to precise, we might bail out on numbers which
// actually would truncate with 64 bit precision.
ASSERT(!CpuFeatures::IsSupported(SSE2) &&
result_state_ <= INT32);
result_state_ = NUMBER;
}
} }
} }