Revision: 24118
Author: [email protected]
Date: Mon Sep 22 12:44:23 2014 UTC
Log: Remove switch-clause limit
BUG=
[email protected]
Review URL: https://codereview.chromium.org/582653002
https://code.google.com/p/v8/source/detail?r=24118
Modified:
/branches/bleeding_edge/src/hydrogen.cc
/branches/bleeding_edge/src/objects.h
=======================================
--- /branches/bleeding_edge/src/hydrogen.cc Mon Sep 22 12:43:40 2014 UTC
+++ /branches/bleeding_edge/src/hydrogen.cc Mon Sep 22 12:44:23 2014 UTC
@@ -4834,14 +4834,9 @@
DCHECK(current_block() != NULL);
DCHECK(current_block()->HasPredecessor());
- // We only optimize switch statements with a bounded number of clauses.
- const int kCaseClauseLimit = 128;
ZoneList<CaseClause*>* clauses = stmt->cases();
int clause_count = clauses->length();
ZoneList<HBasicBlock*> body_blocks(clause_count, zone());
- if (clause_count > kCaseClauseLimit) {
- return Bailout(kSwitchStatementTooManyClauses);
- }
CHECK_ALIVE(VisitForValue(stmt->tag()));
Add<HSimulate>(stmt->EntryId());
=======================================
--- /branches/bleeding_edge/src/objects.h Fri Sep 19 14:59:14 2014 UTC
+++ /branches/bleeding_edge/src/objects.h Mon Sep 22 12:44:23 2014 UTC
@@ -1200,9 +1200,6 @@
V(kSmiSubtractionOverflow, "Smi subtraction
overflow") \
V(kStackAccessBelowStackPointer, "Stack access below stack
pointer") \
V(kStackFrameTypesMustMatch, "Stack frame types must
match") \
-
V(kSwitchStatementMixedOrNonLiteralSwitchLabels,
\
- "SwitchStatement: mixed or non-literal switch
labels") \
- V(kSwitchStatementTooManyClauses, "SwitchStatement: too many
clauses") \
V(kTheCurrentStackPointerIsBelowCsp,
\
"The current stack pointer is below
csp") \
V(kTheInstructionShouldBeALui, "The instruction should be a
lui") \
--
--
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/d/optout.
