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

Issue 1432453003: Fix base register for stack variables in ARM integrated assembler. (Closed)

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

Fix base register for stack variables in ARM integrated assembler. When translating an Variable without a register, the code assumes the variable is on the stack using sp. This is still true. However, if it is a (derived class) StackVariable, the register defined by getBaseRegNum() should be used instead. 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=57586f731ecfd18527434b5256f991bdf2e80192

Patch Set 1 #

Patch Set 2 : Initial patch set. #

Patch Set 3 : Fix minor mistake #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -2 lines) Patch
M src/IceAssemblerARM32.cpp View 1 2 1 chunk +4 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
Karl
5 years, 1 month ago (2015-11-03 22:44:07 UTC) #3
John
lgtm
5 years, 1 month ago (2015-11-03 23:47:51 UTC) #4
Jim Stichnoth
LGTM. If this fixes any of the broken instructions worked around in the other CL, ...
5 years, 1 month ago (2015-11-04 01:04:09 UTC) #5
Karl
Committed patchset #3 (id:40001) manually as 57586f731ecfd18527434b5256f991bdf2e80192 (presubmit successful).
5 years, 1 month ago (2015-11-04 16:10:48 UTC) #6
Karl
5 years, 1 month ago (2015-11-04 16:11:42 UTC) #7
Message was sent while issue was closed.
Unfortunately, this was an independent bug that did not break cross tests, but
is unrelated to the instructions that aren't working.

Powered by Google App Engine
This is Rietveld 408576698