Chromium Code Reviews

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 Stats (+4869 lines, -0 lines)
M Makefile.standalone View 1 chunk +2 lines, -0 lines 0 comments
A src/DartARM32/assembler_arm.h View 1 chunk +1214 lines, -0 lines 0 comments
A src/DartARM32/assembler_arm.cc View 1 chunk +3653 lines, -0 lines 0 comments

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