Reviewers: Toon Verwaest,
Message:
PTAL
Description:
Remove redundant check in LCodeGen::DoBoundsCheck
[email protected]
Please review this at https://chromiumcodereview.appspot.com/12026014/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/ia32/lithium-codegen-ia32.cc
Index: src/ia32/lithium-codegen-ia32.cc
diff --git a/src/ia32/lithium-codegen-ia32.cc
b/src/ia32/lithium-codegen-ia32.cc
index
2ee93e12dec4cb6b78a9568e937a7f9d44a73cf1..e1bff0fa7d15b6de3b1f9b839f1e9db01acc2905
100644
--- a/src/ia32/lithium-codegen-ia32.cc
+++ b/src/ia32/lithium-codegen-ia32.cc
@@ -4154,11 +4154,6 @@ void LCodeGen::DoBoundsCheck(LBoundsCheck* instr) {
}
DeoptimizeIf(below_equal, instr->environment());
} else {
- if (instr->hydrogen()->index()->representation().IsTagged() &&
- !instr->hydrogen()->index()->type().IsSmi()) {
- __ test(ToRegister(instr->index()), Immediate(kSmiTagMask));
- DeoptimizeIf(not_zero, instr->environment());
- }
__ cmp(ToRegister(instr->index()), ToOperand(instr->length()));
DeoptimizeIf(above_equal, instr->environment());
}
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
