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

Issue 1305993005: VM: Simplify irregexp entry stub. (Closed)

Created:
5 years, 3 months ago by Florian Schneider
Modified:
5 years, 3 months ago
Reviewers:
srdjan
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

VM: Simplify irregexp entry stub. Pass on the incoming arguments descriptor when invoking an irregexp function. Avoiding loading an arguments descriptor in the intrinsic code also allows generated regexp code in precompiled code. Loading a new descriptor is not necessary since the arguments are already on the stack in the right order. Use named constants for parameter indices in the regexp assembler. BUG= R=srdjan@google.com Committed: https://github.com/dart-lang/sdk/commit/7e40108ff5175a5aee2c5a5863eaeabe3a6e6b37

Patch Set 1 #

Patch Set 2 : simplify ExecuteMatch #

Unified diffs Side-by-side diffs Delta from patch set Stats (+34 lines, -37 lines) Patch
M runtime/lib/regexp.cc View 1 2 chunks +2 lines, -9 lines 0 comments Download
M runtime/vm/flow_graph_type_propagator.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M runtime/vm/intrinsifier_arm.cc View 1 chunk +0 lines, -2 lines 0 comments Download
M runtime/vm/intrinsifier_arm64.cc View 1 chunk +0 lines, -2 lines 0 comments Download
M runtime/vm/intrinsifier_ia32.cc View 1 chunk +0 lines, -2 lines 0 comments Download
M runtime/vm/intrinsifier_mips.cc View 1 chunk +0 lines, -2 lines 0 comments Download
M runtime/vm/intrinsifier_x64.cc View 1 chunk +0 lines, -3 lines 0 comments Download
M runtime/vm/regexp.cc View 1 1 chunk +12 lines, -4 lines 0 comments Download
M runtime/vm/regexp_assembler.h View 1 chunk +2 lines, -1 line 0 comments Download
M runtime/vm/regexp_assembler_ir.h View 1 1 chunk +1 line, -1 line 0 comments Download
M runtime/vm/regexp_assembler_ir.cc View 1 2 chunks +14 lines, -7 lines 0 comments Download
M runtime/vm/regexp_test.cc View 1 1 chunk +1 line, -4 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
Florian Schneider
5 years, 3 months ago (2015-09-02 08:32:06 UTC) #2
srdjan
lgtm
5 years, 3 months ago (2015-09-02 22:42:53 UTC) #3
Florian Schneider
5 years, 3 months ago (2015-09-04 08:06:07 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
7e40108ff5175a5aee2c5a5863eaeabe3a6e6b37 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698