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

Issue 701233003: Implement bigint intrinsics on arm64. (Closed)

Created:
6 years, 1 month ago by regis
Modified:
6 years, 1 month ago
Reviewers:
zra
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Visibility:
Public.

Description

Implement bigint intrinsics on arm64. Implement umaddl arm64 instruction (assembler, disassembler, simulator, tests). Finish implementing cbz and cbnz arm64 instructions (assembler, tests). R=zra@google.com Committed: https://code.google.com/p/dart/source/detail?r=41608

Patch Set 1 #

Total comments: 1

Patch Set 2 : #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+603 lines, -24 lines) Patch
M runtime/vm/assembler_arm64.h View 1 2 9 chunks +67 lines, -14 lines 0 comments Download
M runtime/vm/assembler_arm64_test.cc View 1 2 2 chunks +91 lines, -0 lines 0 comments Download
M runtime/vm/constants_arm64.h View 1 2 2 chunks +3 lines, -2 lines 0 comments Download
M runtime/vm/disassembler_arm64.cc View 1 2 1 chunk +3 lines, -0 lines 0 comments Download
M runtime/vm/intrinsifier_arm.cc View 1 2 4 chunks +8 lines, -2 lines 0 comments Download
M runtime/vm/intrinsifier_arm64.cc View 1 2 1 chunk +422 lines, -6 lines 0 comments Download
M runtime/vm/method_recognizer.h View 1 2 1 chunk +1 line, -0 lines 0 comments Download
M runtime/vm/simulator_arm64.cc View 1 2 1 chunk +8 lines, -0 lines 0 comments Download

Messages

Total messages: 10 (1 generated)
regis
I'd also be interested to check if it works on the hardware. Thanks! -- Regis
6 years, 1 month ago (2014-11-05 21:28:51 UTC) #2
zra
https://codereview.chromium.org/701233003/diff/1/runtime/vm/assembler_arm64.h File runtime/vm/assembler_arm64.h (right): https://codereview.chromium.org/701233003/diff/1/runtime/vm/assembler_arm64.h#newcode1675 runtime/vm/assembler_arm64.h:1675: void EmitCompareAndBranch(CompareAndBranchOp op, Register rt, You also need to ...
6 years, 1 month ago (2014-11-06 00:30:59 UTC) #3
regis
On 2014/11/06 00:30:59, zra wrote: > https://codereview.chromium.org/701233003/diff/1/runtime/vm/assembler_arm64.h > File runtime/vm/assembler_arm64.h (right): > > https://codereview.chromium.org/701233003/diff/1/runtime/vm/assembler_arm64.h#newcode1675 > ...
6 years, 1 month ago (2014-11-06 18:43:51 UTC) #4
regis
On 2014/11/06 18:43:51, regis wrote: > On 2014/11/06 00:30:59, zra wrote: > > https://codereview.chromium.org/701233003/diff/1/runtime/vm/assembler_arm64.h > ...
6 years, 1 month ago (2014-11-07 17:01:45 UTC) #5
zra
lgtm
6 years, 1 month ago (2014-11-07 17:02:16 UTC) #6
regis
Thanks!
6 years, 1 month ago (2014-11-07 17:19:36 UTC) #7
regis
Committed patchset #3 (id:40001) manually as r41608 (presubmit successful).
6 years, 1 month ago (2014-11-07 17:19:50 UTC) #8
zra
I verified that the new assembler tests and the RSA benchmark are working on hardware.
6 years, 1 month ago (2014-11-07 23:22:11 UTC) #9
regis
6 years, 1 month ago (2014-11-07 23:29:28 UTC) #10
Message was sent while issue was closed.
Thanks for verifying!

Powered by Google App Engine
This is Rietveld 408576698