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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1161543005: Subzero: Changes needed for LLVM 3.7 integration. (Closed)

Created:
5 years, 6 months ago by Jim Stichnoth
Modified:
5 years, 6 months ago
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: Changes needed for LLVM 3.7 integration. 1. Change Makefile.standalone from 3.6 to 3.7. 2. Update to new load instruction .ll syntax. This includes changing InstLoad::dump() to match. BUG= none R=jvoung@chromium.org Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=e5b58fbebc37b8c2a11317afdb3dc82684147207

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+140 lines, -140 lines) Patch
M Makefile.standalone View 2 chunks +2 lines, -2 lines 0 comments Download
M src/IceInst.cpp View 1 chunk +1 line, -1 line 0 comments Download
M tests_lit/assembler/x86/opcode_register_encodings.ll View 2 chunks +4 lines, -4 lines 0 comments Download
M tests_lit/llvm2ice_tests/64bit.pnacl.ll View 1 chunk +1 line, -1 line 0 comments Download
M tests_lit/llvm2ice_tests/8bit.pnacl.ll View 4 chunks +4 lines, -4 lines 0 comments Download
M tests_lit/llvm2ice_tests/addr-opt-multi-def-var.ll View 3 chunks +4 lines, -4 lines 0 comments Download
M tests_lit/llvm2ice_tests/address-mode-opt.ll View 12 chunks +12 lines, -12 lines 0 comments Download
M tests_lit/llvm2ice_tests/callindirect.pnacl.ll View 1 chunk +1 line, -1 line 0 comments Download
M tests_lit/llvm2ice_tests/convert.ll View 8 chunks +8 lines, -8 lines 0 comments Download
M tests_lit/llvm2ice_tests/fp.pnacl.ll View 2 chunks +2 lines, -2 lines 0 comments Download
M tests_lit/llvm2ice_tests/ias-multi-reloc.ll View 2 chunks +2 lines, -2 lines 0 comments Download
M tests_lit/llvm2ice_tests/load.ll View 1 chunk +8 lines, -8 lines 0 comments Download
M tests_lit/llvm2ice_tests/load_cast.ll View 20 chunks +20 lines, -20 lines 0 comments Download
M tests_lit/llvm2ice_tests/nacl-atomic-fence-all.ll View 5 chunks +12 lines, -12 lines 0 comments Download
M tests_lit/llvm2ice_tests/nacl-other-intrinsics.ll View 3 chunks +3 lines, -3 lines 0 comments Download
M tests_lit/llvm2ice_tests/phi.ll View 1 chunk +1 line, -1 line 0 comments Download
M tests_lit/llvm2ice_tests/regalloc_evict_non_overlap.ll View 2 chunks +3 lines, -3 lines 0 comments Download
M tests_lit/llvm2ice_tests/shift.ll View 2 chunks +2 lines, -2 lines 0 comments Download
M tests_lit/llvm2ice_tests/simple-loop.ll View 1 chunk +1 line, -1 line 0 comments Download
M tests_lit/llvm2ice_tests/struct-arith.pnacl.ll View 1 chunk +9 lines, -9 lines 0 comments Download
M tests_lit/llvm2ice_tests/vector-align.ll View 7 chunks +7 lines, -7 lines 0 comments Download
M tests_lit/reader_tests/extern_globals.ll View 9 chunks +9 lines, -9 lines 0 comments Download
M tests_lit/reader_tests/load.ll View 5 chunks +24 lines, -24 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
Jim Stichnoth
5 years, 6 months ago (2015-06-01 21:41:50 UTC) #2
jvoung (off chromium)
LGTM thanks
5 years, 6 months ago (2015-06-01 21:44:59 UTC) #3
Jim Stichnoth
5 years, 6 months ago (2015-06-01 22:17:24 UTC) #4
Message was sent while issue was closed. 
Committed patchset #1 (id:1) manually as
e5b58fbebc37b8c2a11317afdb3dc82684147207 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698