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

Issue 2376893003: Emit switchable instance calls when we don't know the complete set of receiver classes for a call s… (Closed)

Created:
4 years, 2 months ago by rmacnak
Modified:
4 years, 2 months ago
Reviewers:
Florian Schneider
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Emit switchable instance calls when we don't know the complete set of receiver classes for a call site. A switchable call in the monomorphic state is now basically the same cost as a test and call, so this is as performant without ht code size cost of the slow path megemorphic call. Flutter gallery before: VMIsolate(CodeSize): 114052 Isolate(CodeSize): 2250283 ReadOnlyData(CodeSize): 1744760 Instructions(CodeSize): 5679488 Total(CodeSize): 9788583 After: VMIsolate(CodeSize): 114085 Isolate(CodeSize): 2047224 ReadOnlyData(CodeSize): 1666808 Instructions(CodeSize): 5405424 -4.8% Total(CodeSize): 9233541 -5.6% R=fschneider@google.com Committed: https://github.com/dart-lang/sdk/commit/271483911fb26294d215cec8e567c43558c78da2

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -110 lines) Patch
M runtime/vm/flow_graph_compiler.h View 1 chunk +0 lines, -31 lines 0 comments Download
M runtime/vm/flow_graph_compiler.cc View 1 chunk +2 lines, -14 lines 0 comments Download
M runtime/vm/flow_graph_compiler_arm.cc View 1 chunk +0 lines, -12 lines 0 comments Download
M runtime/vm/flow_graph_compiler_arm64.cc View 1 chunk +0 lines, -12 lines 0 comments Download
M runtime/vm/flow_graph_compiler_dbc.cc View 1 chunk +0 lines, -5 lines 0 comments Download
M runtime/vm/flow_graph_compiler_ia32.cc View 1 chunk +0 lines, -12 lines 0 comments Download
M runtime/vm/flow_graph_compiler_mips.cc View 1 chunk +0 lines, -12 lines 0 comments Download
M runtime/vm/flow_graph_compiler_x64.cc View 1 chunk +0 lines, -12 lines 0 comments Download

Messages

Total messages: 8 (4 generated)
rmacnak
4 years, 2 months ago (2016-09-29 00:49:26 UTC) #4
rmacnak
Golem: FluidMotion (Intel Core 2) -4.393% 0.73 0.76 ParticleSystemUpdate (Intel Core i5) -1.543% 27.20 27.63 ...
4 years, 2 months ago (2016-09-30 21:51:11 UTC) #5
Florian Schneider
Lgtm
4 years, 2 months ago (2016-09-30 23:54:56 UTC) #6
rmacnak
4 years, 2 months ago (2016-10-01 00:34:44 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
271483911fb26294d215cec8e567c43558c78da2 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698