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

Issue 300003006: Add fast path to ListGetAsBytes for byte-size ExternalTypedData. (Closed)

Created:
6 years, 7 months ago by Florian Schneider
Modified:
6 years, 7 months ago
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Visibility:
Public.

Description

Add fast path to ListGetAsBytes for byte-size ExternalTypedData. This speeds up deserializing snapshots which come in as ExternalUint8Array. This API function should be generalized to be fast for all built-in array types in a future CL. R=ajohnsen@google.com, iposva@google.com Committed: https://code.google.com/p/dart/source/detail?r=36671

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : fixed assertion #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -3 lines) Patch
M runtime/vm/dart_api_impl.cc View 1 2 2 chunks +12 lines, -3 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Florian Schneider
6 years, 7 months ago (2014-05-27 12:39:40 UTC) #1
Anders Johnsen
lgtm, nice catch
6 years, 7 months ago (2014-05-27 12:41:26 UTC) #2
Ivan Posva
LGTM -ip
6 years, 7 months ago (2014-05-27 13:05:24 UTC) #3
Florian Schneider
6 years, 7 months ago (2014-05-27 13:05:59 UTC) #4
Message was sent while issue was closed.
Committed patchset #3 manually as r36671 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698