Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(27)

Issue 370553002: ARM64: use direct conditional branches for regexp (Closed)

Created:
6 years, 5 months ago by vincent.belliard
Modified:
6 years, 5 months ago
Reviewers:
Benedikt Meurer, ulan
CC:
v8-dev
Visibility:
Public.

Description

ARM64: use direct conditional branches for regexp R=ulan@chromium.org, bmeurer@chromium.org BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -12 lines) Patch
M src/arm64/regexp-macro-assembler-arm64.cc View 2 chunks +3 lines, -12 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
vincent.belliard
With veneers, we can now do direct conditional branches. If applied with the Adr patch, ...
6 years, 5 months ago (2014-07-03 11:04:32 UTC) #1
ulan
On 2014/07/03 11:04:32, vincent.belliard.arm wrote: > With veneers, we can now do direct conditional branches. ...
6 years, 5 months ago (2014-07-03 11:19:27 UTC) #2
vincent.belliard
> This is the same as https://codereview.chromium.org/236163006/, where Alexandre > observed ~8% performance penalty on ...
6 years, 5 months ago (2014-07-03 11:30:57 UTC) #3
vincent.belliard
6 years, 5 months ago (2014-07-03 13:21:45 UTC) #4
committed r22199

Powered by Google App Engine
This is Rietveld 408576698