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

Issue 12310117: Fix breakage (Closed)

Created:
7 years, 9 months ago by Cutch
Modified:
7 years, 9 months ago
Reviewers:
srdjan
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Fix dart2js breakage from Simd landing Committed: https://code.google.com/p/dart/source/detail?r=19004

Patch Set 1 #

Total comments: 3

Patch Set 2 : Fix whitespace #

Patch Set 3 : Srdjan's review #

Unified diffs Side-by-side diffs Delta from patch set Stats (+35 lines, -0 lines) Patch
M runtime/tests/vm/vm.status View 1 2 1 chunk +1 line, -0 lines 0 comments Download
M sdk/lib/_internal/compiler/implementation/lib/scalarlist_patch.dart View 1 2 1 chunk +34 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Cutch
7 years, 9 months ago (2013-02-25 21:43:50 UTC) #1
Cutch
7 years, 9 months ago (2013-02-25 21:45:19 UTC) #2
srdjan
LGTM https://codereview.chromium.org/12310117/diff/1/runtime/tests/vm/vm.status File runtime/tests/vm/vm.status (right): https://codereview.chromium.org/12310117/diff/1/runtime/tests/vm/vm.status#newcode36 runtime/tests/vm/vm.status:36: dart/simd128float32_array_test: Skip # compilers not aware of byte ...
7 years, 9 months ago (2013-02-25 21:46:22 UTC) #3
Cutch
7 years, 9 months ago (2013-02-25 21:50:09 UTC) #4
Message was sent while issue was closed.
Committed patchset #3 manually as r19004 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698