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

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 Delta from patch set Stats (+213 lines, -42 lines) Patch
M src/IceInst.h View 4 chunks +3 lines, -4 lines 1 comment Download
M src/IceInstX8632.h View 4 chunks +6 lines, -0 lines 0 comments Download
M src/IceInstX8632.cpp View 1 2 3 6 chunks +105 lines, -24 lines 2 comments Download
M src/assembler_ia32.h View 1 chunk +0 lines, -1 line 0 comments Download
M src/assembler_ia32.cpp View 1 chunk +0 lines, -7 lines 0 comments Download
M tests_lit/llvm2ice_tests/64bit.pnacl.ll View 2 chunks +4 lines, -6 lines 0 comments Download
M tests_lit/llvm2ice_tests/8bit.pnacl.ll View 2 chunks +55 lines, -0 lines 0 comments Download
M tests_lit/llvm2ice_tests/callindirect.pnacl.ll View 1 2 3 1 chunk +40 lines, -0 lines 0 comments Download

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
This is Rietveld 408576698