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

Issue 1382063002: Subzero. Adds I64 register pairs for ARM32. (Closed)

Created:
5 years, 2 months ago by John
Modified:
5 years, 2 months ago
Reviewers:
Jim Stichnoth, Karl, sehr
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

Subzero. Adds I64 register pairs for ARM32. This is in preparation for llvm.nacl.atomic.* lowerings. atomic i64 loads and stores require their operands to be consecutive registers starting at an even register that is not r14. BUG= https://code.google.com/p/nativeclient/issues/detail?id=4076 R=kschimpf@google.com Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=ed2c06b20a04fc07f0a54b7ce01f19c98bd5a3ab

Patch Set 1 #

Patch Set 2 : merge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+192 lines, -138 lines) Patch
M src/IceInstARM32.def View 14 chunks +170 lines, -128 lines 0 comments Download
M src/IceRegistersARM32.h View 6 chunks +15 lines, -5 lines 0 comments Download
M src/IceTargetLoweringARM32.cpp View 5 chunks +7 lines, -5 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
John
5 years, 2 months ago (2015-10-01 20:53:46 UTC) #2
John
5 years, 2 months ago (2015-10-01 22:14:24 UTC) #4
Karl
lgtm
5 years, 2 months ago (2015-10-01 22:25:41 UTC) #5
John
Committed patchset #2 (id:20001) manually as ed2c06b20a04fc07f0a54b7ce01f19c98bd5a3ab (presubmit successful).
5 years, 2 months ago (2015-10-01 22:27:19 UTC) #6
Jim Stichnoth
5 years, 2 months ago (2015-10-01 23:16:45 UTC) #7
Message was sent while issue was closed.
also lgtm

Powered by Google App Engine
This is Rietveld 408576698