Chromium Code Reviews

Issue 647193003: emitIAS for store and indirect calls. (Closed)

Created:
6 years, 2 months ago by jvoung (off chromium)
Modified:
6 years, 2 months ago
Reviewers:
Jim Stichnoth
CC:
native-client-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Visibility:
Public.

Description

emitIAS for store and indirect calls. We can't do direct calls via the .long sym hack, since that is normally for an absolute relocation, but calls are expecting relative relocations (except for reg/mem forms). Nop-out the InstFake emitIAS methods. Remove the generic dispatcher that redirects emitIAS() to emit(), since only branches and labels are left. BUG=none R=stichnot@chromium.org Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=198b294823bc12bfcf3addb11dc26d4772e2cb8f

Patch Set 1 #

Patch Set 2 : stuff #

Patch Set 3 : grammar #

Patch Set 4 : stuff #

Total comments: 3
Unified diffs Side-by-side diffs Stats (+213 lines, -42 lines)
M src/IceInst.h View 4 chunks +3 lines, -4 lines 1 comment
M src/IceInstX8632.h View 4 chunks +6 lines, -0 lines 0 comments
M src/IceInstX8632.cpp View 6 chunks +105 lines, -24 lines 2 comments
M src/assembler_ia32.h View 1 chunk +0 lines, -1 line 0 comments
M src/assembler_ia32.cpp View 1 chunk +0 lines, -7 lines 0 comments
M tests_lit/llvm2ice_tests/64bit.pnacl.ll View 2 chunks +4 lines, -6 lines 0 comments
M tests_lit/llvm2ice_tests/8bit.pnacl.ll View 2 chunks +55 lines, -0 lines 0 comments
M tests_lit/llvm2ice_tests/callindirect.pnacl.ll View 1 chunk +40 lines, -0 lines 0 comments

Messages

Total messages: 5 (1 generated)
jvoung (off chromium)
6 years, 2 months ago (2014-10-16 01:52:05 UTC) #2
Jim Stichnoth
lgtm https://codereview.chromium.org/647193003/diff/60001/src/IceInst.h File src/IceInst.h (left): https://codereview.chromium.org/647193003/diff/60001/src/IceInst.h#oldcode814 src/IceInst.h:814: void emitIAS(const Cfg *Func) const override { emit(Func); ...
6 years, 2 months ago (2014-10-16 02:48:16 UTC) #3
jvoung (off chromium)
https://codereview.chromium.org/647193003/diff/60001/src/IceInstX8632.cpp File src/IceInstX8632.cpp (right): https://codereview.chromium.org/647193003/diff/60001/src/IceInstX8632.cpp#newcode466 src/IceInstX8632.cpp:466: assert(CR->getOffset() == 0 && "We only support calling a ...
6 years, 2 months ago (2014-10-16 15:22:12 UTC) #4
jvoung (off chromium)
6 years, 2 months ago (2014-10-16 16:40:08 UTC) #5
Message was sent while issue was closed.
Committed patchset #4 (id:60001) manually as
198b294823bc12bfcf3addb11dc26d4772e2cb8f (presubmit successful).

Powered by Google App Engine