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

Issue 19493004: Adds more SIMD instructions for ARM. (Closed)

Created:
7 years, 5 months ago by zra
Modified:
7 years, 5 months ago
Reviewers:
regis
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Adds more SIMD instructions for ARM. Adds the vnegqs, vabsqs, and vandq instructions. Also reverts the increase in the number of floating point registers since it is causing crashes on hardware. R=regis@google.com Committed: https://code.google.com/p/dart/source/detail?r=25320

Patch Set 1 #

Patch Set 2 : #

Total comments: 7

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+163 lines, -3 lines) Patch
M runtime/vm/assembler_arm.h View 2 chunks +3 lines, -0 lines 0 comments Download
M runtime/vm/assembler_arm.cc View 1 2 chunks +17 lines, -0 lines 0 comments Download
M runtime/vm/assembler_arm_test.cc View 1 2 chunks +100 lines, -0 lines 0 comments Download
M runtime/vm/constants_arm.h View 1 2 2 chunks +7 lines, -1 line 0 comments Download
M runtime/vm/disassembler_arm.cc View 1 chunk +11 lines, -0 lines 0 comments Download
M runtime/vm/simulator_arm.cc View 1 2 3 chunks +25 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
zra
7 years, 5 months ago (2013-07-22 21:14:45 UTC) #1
regis
LGTM https://codereview.chromium.org/19493004/diff/8001/runtime/vm/constants_arm.h File runtime/vm/constants_arm.h (right): https://codereview.chromium.org/19493004/diff/8001/runtime/vm/constants_arm.h#newcode15 runtime/vm/constants_arm.h:15: #define VFPv3_D16 After you figure out the issue ...
7 years, 5 months ago (2013-07-22 22:47:11 UTC) #2
zra
Thanks! I will commit this change with VFPv3_D16 defined, and find/fix the VFPv3_D32 bug in ...
7 years, 5 months ago (2013-07-22 23:20:09 UTC) #3
zra
7 years, 5 months ago (2013-07-22 23:26:16 UTC) #4
Message was sent while issue was closed.
Committed patchset #3 manually as r25320 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698