Chromium Code Reviews

Issue 1667553002: Subzero: change Inst->Instr in many place to follow newer convention. (Closed)

Created:
4 years, 10 months ago by rkotlerimgtec
Modified:
4 years, 10 months ago
Reviewers:
kschimpf, Jim Stichnoth, John
CC:
native-client-reviews_googlegroups.com, rich.fuhler_imgtec.com
Base URL:
https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Following the newer convention enhances readability and in addition this is a prelude to some macro changes in unimplemented I would like to make in order to simplify that code. BUG= R=stichnot@chromium.org Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=04bca5a57784a630ffb31c405f2da1f1b43d086e

Patch Set 1 #

Patch Set 2 : A few more Inst->Instr #

Total comments: 2

Patch Set 3 : changes suggested by stichnot #

Unified diffs Side-by-side diffs Stats (+79 lines, -79 lines)
M src/IceTargetLoweringMIPS32.h View 2 chunks +20 lines, -20 lines 0 comments
M src/IceTargetLoweringMIPS32.cpp View 13 chunks +59 lines, -59 lines 0 comments

Messages

Total messages: 9 (3 generated)
rkotlerimgtec
4 years, 10 months ago (2016-02-03 02:57:36 UTC) #3
rkotlerimgtec
Forgot to hit save and missed a few of the Inst->Instr changes.
4 years, 10 months ago (2016-02-03 04:25:47 UTC) #4
Jim Stichnoth
Might as well also change the instances in IceTargetLoweringMIPS32.h . https://codereview.chromium.org/1667553002/diff/20001/src/IceTargetLoweringMIPS32.cpp File src/IceTargetLoweringMIPS32.cpp (right): https://codereview.chromium.org/1667553002/diff/20001/src/IceTargetLoweringMIPS32.cpp#newcode1088 ...
4 years, 10 months ago (2016-02-03 05:03:23 UTC) #5
rkotlerimgtec
https://codereview.chromium.org/1667553002/diff/20001/src/IceTargetLoweringMIPS32.cpp File src/IceTargetLoweringMIPS32.cpp (right): https://codereview.chromium.org/1667553002/diff/20001/src/IceTargetLoweringMIPS32.cpp#newcode1088 src/IceTargetLoweringMIPS32.cpp:1088: void TargetMIPS32::lowerPhi(const InstPhi * /*Inst*/) { On 2016/02/03 05:03:23, ...
4 years, 10 months ago (2016-02-03 22:06:39 UTC) #6
Jim Stichnoth
lgtm
4 years, 10 months ago (2016-02-03 22:40:14 UTC) #7
Jim Stichnoth
4 years, 10 months ago (2016-02-03 22:40:52 UTC) #9
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
04bca5a57784a630ffb31c405f2da1f1b43d086e (presubmit successful).

Powered by Google App Engine