From 00170e434b617ca8e463f11df5b49cae86c6bec4 Mon Sep 17 00:00:00 2001 From: "yangguo@chromium.org" Date: Thu, 20 Mar 2014 12:34:44 +0000 Subject: [PATCH] Use HBoundsCheck to check string length. R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/206183003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@20120 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- src/hydrogen.cc | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/hydrogen.cc b/src/hydrogen.cc index f6e22d8..282fae7 100644 --- a/src/hydrogen.cc +++ b/src/hydrogen.cc @@ -1793,11 +1793,8 @@ HValue* HGraphBuilder::BuildAddStringLengths(HValue* left_length, HValue* right_length) { // Compute the combined string length and check against max string length. HValue* length = AddUncasted(left_length, right_length); - IfBuilder if_nooverflow(this); - if_nooverflow.If( - length, Add(String::kMaxLength), Token::LTE); - if_nooverflow.Then(); - if_nooverflow.ElseDeopt("String length exceeds limit"); + HValue* max_length = Add(String::kMaxLength); + Add(length, max_length); return length; } -- 2.7.4