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

Issue 1394613002: Create local copy of Dart assembler code. (Closed)

Created:
5 years, 2 months ago by Karl
Modified:
5 years, 2 months ago
Reviewers:
Jim Stichnoth, John
CC:
native-client-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Create local copy of Dart assembler code. Creates a local version of the Dart assembler code, before being merged into our code base. The goal of these files is to track code as it is moved from the Dart implementation into our code base. BUG= https://code.google.com/p/nativeclient/issues/detail?id=4334 R=jpp@chromium.org, stichnot@chromium.org Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=3e53dc99c9f61fc1f65e85735220f0b03d2487ca

Patch Set 1 #

Patch Set 2 : Blacklist ARM32 Dart files. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4869 lines, -0 lines) Patch
M Makefile.standalone View 1 1 chunk +2 lines, -0 lines 0 comments Download
A src/DartARM32/assembler_arm.h View 1 chunk +1214 lines, -0 lines 0 comments Download
A src/DartARM32/assembler_arm.cc View 1 chunk +3653 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
Karl
5 years, 2 months ago (2015-10-07 18:07:38 UTC) #2
John
lgtm
5 years, 2 months ago (2015-10-07 19:24:38 UTC) #3
Jim Stichnoth
Also LGTM. Can you add these files to FORMAT_BLACKLIST in Makefile.standalone so that "make format" ...
5 years, 2 months ago (2015-10-07 19:50:01 UTC) #4
Karl
5 years, 2 months ago (2015-10-07 20:24:07 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
3e53dc99c9f61fc1f65e85735220f0b03d2487ca (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698