From: baptiste.afsa@arm.com Date: Wed, 19 Feb 2014 09:37:07 +0000 (+0000) Subject: A64: Replace a Cmp/Branch sequence by a conditional compare. X-Git-Tag: upstream/4.7.83~10626 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=53f2aefe088c6b6fee0dfe5eee360de12065c1c8;p=platform%2Fupstream%2Fv8.git A64: Replace a Cmp/Branch sequence by a conditional compare. R=jochen@chromium.org, ulan@chromium.org Review URL: https://codereview.chromium.org/169403003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@19472 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- diff --git a/src/a64/regexp-macro-assembler-a64.cc b/src/a64/regexp-macro-assembler-a64.cc index 00558c0..b162b38 100644 --- a/src/a64/regexp-macro-assembler-a64.cc +++ b/src/a64/regexp-macro-assembler-a64.cc @@ -363,11 +363,9 @@ void RegExpMacroAssemblerA64::CheckNotBackReferenceIgnoreCase( __ B(ls, &loop_check); // In range 'a'-'z'. // Latin-1: Check for values in range [224,254] but not 247. __ Sub(w10, w10, 224 - 'a'); - // TODO(jbramley): Use Ccmp here. __ Cmp(w10, 254 - 224); - __ B(hi, &fail); // Weren't Latin-1 letters. - __ Cmp(w10, 247 - 224); // Check for 247. - __ B(eq, &fail); + __ Ccmp(w10, 247 - 224, ZFlag, ls); // Check for 247. + __ B(eq, &fail); // Weren't Latin-1 letters. __ Bind(&loop_check); __ Cmp(capture_start_address, capture_end_addresss);