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

Issue 11280121: Intrinsify ExternalUint8Array.[]. (Closed)

Created:
8 years ago by Mads Ager (google)
Modified:
8 years ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Intrinsify ExternalUint8Array.[]. This helps the HTTP parser quite a bit. I was playing around with it and thought I might as well upload it for review. Feel free to take over the changelist if I'm doing this wrong. In that case you should see this as a request for intrinsification of ExternalUint8Arrays. :-) R=fschneider@google.com,srdjan@google.com BUG= Committed: https://code.google.com/p/dart/source/detail?r=15200

Patch Set 1 #

Total comments: 4

Patch Set 2 : Use OFFSET_OF #

Patch Set 3 : Consistently use OFFSET_OF #

Unified diffs Side-by-side diffs Delta from patch set Stats (+47 lines, -10 lines) Patch
M runtime/vm/intrinsifier.h View 1 chunk +1 line, -0 lines 0 comments Download
M runtime/vm/intrinsifier_ia32.cc View 1 chunk +14 lines, -0 lines 0 comments Download
M runtime/vm/intrinsifier_x64.cc View 1 chunk +14 lines, -0 lines 0 comments Download
M runtime/vm/object.h View 1 2 11 chunks +14 lines, -10 lines 0 comments Download
M runtime/vm/raw_object.h View 1 2 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Mads Ager (google)
8 years ago (2012-11-21 14:52:00 UTC) #1
Florian Schneider
LGTM. Of course, we also want an IL instruction in the optimizing compiler eventually. https://codereview.chromium.org/11280121/diff/1/runtime/vm/object.h ...
8 years ago (2012-11-21 15:15:17 UTC) #2
Mads Ager (google)
Thanks Florian. I change all of the data_offset getters for typed arrays to use OFFSET_OF. ...
8 years ago (2012-11-21 15:31:34 UTC) #3
srdjan
8 years ago (2012-11-21 15:32:44 UTC) #4
What Florian wrote and LGTM

Powered by Google App Engine
This is Rietveld 408576698