- 27 Dec, 2016 2 commits
-
-
Jim Stichnoth authored
LLVM trunk commit a5479197371e169bc676e0e570f2a994329f8773 split include/llvm/Bitcode/ReaderWriter.h into two separate header files. This CL hacks it back, in a sense. With this change (along with the rest of the recent fixes), Subzero builds against the latest LLVM trunk - 759dd39154f0bbf1adc87bf613c95f9564f64aa8 at this writing. BUG= none R=kschimpf@google.com Review-Url: https://codereview.chromium.org/2604753002 .
-
Jim Stichnoth authored
LLVM commit 4ac35e82723d4bdbc094f73b2d33aaaa86a00d6a removed some code that Subzero still needs, specifically StreamingMemoryObject and its parents. Deal with this by adding those files back to the Subzero repo. This allows Subzero to build through LLVM c75e3c2fd2bb9ca010f4e1c32acbd142adc32c7f. BUG= none R=kschimpf@google.com Review-Url: https://codereview.chromium.org/2605653002 .
-
- 26 Dec, 2016 2 commits
-
-
Jim Stichnoth authored
This allows Subzero to build through LLVM e8516587a2604386a8faaab28c410663c2ec884c. 1. Mirror an internal change to the llvm::cl implementation. 2. Something changed in llvm::format() that now requires an explicit conversion from RegNumT. BUG= none R=kschimpf@google.com Review-Url: https://codereview.chromium.org/2602713002 .
-
Jim Stichnoth authored
The existing code gives a build error against LLVM trunk. Actually, the latest LLVM trunk causes a lot more Subzero build problems, but if we roll LLVM back to 1d79fff9e65e77f84bf80c2cf4f0155bd167c90d, everything builds except for this one issue. BUG= none R=kschimpf@google.com Review-Url: https://codereview.chromium.org/2598263002 .
-
- 24 Dec, 2016 1 commit
-
-
Jim Stichnoth authored
Based on build failure messages in https://build.chromium.org/p/tryserver.nacl/builders/nacl-toolchain-win7-pnacl-x86_64/builds/3609/steps/llvm_i686_w64_mingw32%20%28build%29/logs/stdio . BUG= none R=jpp@chromium.org Review URL: https://codereview.chromium.org/2601653002 .
-
- 23 Dec, 2016 1 commit
-
-
Jim Stichnoth authored
The movzx operand must be a register or memory operand. An immediate operand is not allowed. BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=4384 R=jpp@chromium.org Review URL: https://codereview.chromium.org/2598153002 .
-
- 09 Dec, 2016 1 commit
-
-
Nicolas Capens authored
The arguments get legalized to Reg or Mem, so we can allow constants as well (including undef values). This change makes all instruction's source arguments Ice::Operands. BUG=swiftshader:24 Change-Id: I1659cdfdb1b8a12c4acc7c473211d8a67bfd5868 Reviewed-on: https://chromium-review.googlesource.com/418504Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
- 30 Nov, 2016 1 commit
-
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: I7e10342fa1ef9bce22bc8c445240fc34a68e8f47 Reviewed-on: https://chromium-review.googlesource.com/414992Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
- 28 Nov, 2016 2 commits
-
-
Nicolas Capens authored
Fixes hitting a (benign) assert in replaceSource(). Change-Id: I7f984d484133e619717d004f20cd671a54473185 Reviewed-on: https://chromium-review.googlesource.com/414490Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
Jaydeep Patil authored
This patch fixes size of arguments on stack R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2533563002 . Patch from Jaydeep Patil <jaydeep.patil@imgtec.com>.
-
- 24 Nov, 2016 1 commit
-
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: If608ab4903d97daa0ad342d02f496ac3fa6471d9 Reviewed-on: https://chromium-review.googlesource.com/414389Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
- 23 Nov, 2016 1 commit
-
-
Nicolas Capens authored
Change-Id: I469a7ddaa658d79fc491112b63972bd9b056689d Reviewed-on: https://chromium-review.googlesource.com/414186Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
- 22 Nov, 2016 1 commit
-
-
Stefan Maksimovic authored
R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2519863002 . Patch from Stefan Maksimovic <makdstefan@gmail.com>.
-
- 18 Nov, 2016 1 commit
-
-
Sagar Thakur authored
R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2504253002 . Patch from Sagar Thakur <sagar.thakur@imgtec.com>.
-
- 10 Nov, 2016 1 commit
-
-
Stefan Maksimovic authored
Initial patch regarding the Subzero MIPS32 sandboxing stage. At the moment, the results of the crosstests with vector tests disabled are as follows: ASM mode: 19 passing / 5 failing test_bitmanip: O2 test_calling_conv: Om1, O2 test_sync_atomic: Om1, O2 ELF mode: 15 passing / 9 failing test_bitmanip: O2 test_calling_conv: Om1, O2 test_global: Om1, O2 test_stacksave: Om1, O2 test_sync_atomic: Om1, O2 R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2482123002 . Patch from Stefan Maksimovic <makdstefan@gmail.com>.
-
- 09 Nov, 2016 1 commit
-
-
Nicolas Capens authored
BUG=swiftshader:9 Change-Id: I779abfe7775632e1108e9d608bf21a63c8cefe9e Reviewed-on: https://chromium-review.googlesource.com/407882Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 07 Nov, 2016 1 commit
-
-
Jaydeep Patil authored
This patch optimizes code generation of instructions with 16-bit immediate operands R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2478113003 . Patch from Jaydeep Patil <jaydeep.patil@imgtec.com>.
-
- 04 Nov, 2016 1 commit
-
-
Jaydeep Patil authored
The patch generates MIPS.abiflags section. This section contains a versioned data structure with essential information required for loader to determine the requirements of the application. R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2471883005 . Patch from Jaydeep Patil <jaydeep.patil@imgtec.com>.
-
- 03 Nov, 2016 1 commit
-
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: I8e53f2fdb8208f8be0f4cdff3241b4a5efe9bc8a Reviewed-on: https://chromium-review.googlesource.com/404352Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
- 02 Nov, 2016 2 commits
-
-
Stefan Maksimovic authored
Implemets Stacksave/Stackrestore; test_stacksave runs successfully when jal implementation is present, both in forceasm as well as in elf mode R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2455933002 . Patch from Stefan Maksimovic <makdstefan@gmail.com>.
-
Jaydeep Patil authored
The patch fixes legalizeToReg issues in vector code generation. The patch also generates JALR for pointer to function and corrects encoding of FP conditional move instruction. R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2468133002 . Patch from Jaydeep Patil <jaydeep.patil@imgtec.com>.
-
- 01 Nov, 2016 3 commits
-
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: Ic120eddd1761e33b7d76bf3ed8ec5ca74634f958 Reviewed-on: https://chromium-review.googlesource.com/403477Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: Ib822b50c0a14e5ebc114db9759cbeecbb9f7a3c1 Reviewed-on: https://chromium-review.googlesource.com/403472Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: Ib89d628c85696c20a249b8810cd357a292d10402 Reviewed-on: https://chromium-review.googlesource.com/405293Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 31 Oct, 2016 1 commit
-
-
Stefan Maksimovic authored
R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2448193008 . Patch from Stefan Maksimovic <makdstefan@gmail.com>.
-
- 28 Oct, 2016 3 commits
-
-
Nicolas Capens authored
Also, their priority is lowered so that registers which are scratch on both Unix and Windows are preferred by the register allocator. BUG=swiftshader:22 Change-Id: Id55d8c8b8c106947e3041a082099069d7c6c6ed0 Reviewed-on: https://chromium-review.googlesource.com/404503Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Tested-by:
Jim Stichnoth <stichnot@chromium.org>
-
Jaydeep Patil authored
The patch generate Hi, Lo, Jal and data relocations. Instruction encoding of instructions like ldc1, sdc1 etc. has been added. Following tests from cross-test framework were tested successfully: (non-vector, OBJ mode, Om1, O2) mem_intrin TotalTests=114300 Passes=114300 Failures=0 simple_loop TotalTests=102 Passes=102 Failures=0 test_arith TotalTests=49489704 Passes=49489704 Failures=0 test_bitmanip TotalTests=1200 Passes=1200 Failures=0 test_cast TotalTests=7444 Passes=7444 Failures=0 test_fcmp TotalTests=123904 Passes=123904 Failures=0 test_global TotalTests=270 Passes=270 Failures=0 test_icmp TotalTests=3341520 Passes=3341520 Failures=0 test_strengthreduce TotalTests=240 Passes=240 Failures=0 Following tests are disabled as they are either all-vectors or contain unimplemented intrinsic lowering: test_calling_conv test_select test_stacksave test_sync_atomic test_vector_ops R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2446273003 . Patch from Jaydeep Patil <jaydeep.patil@imgtec.com>.
-
Stefan Maksimovic authored
R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2433243003 . Patch from Stefan Maksimovic <makdstefan@gmail.com>.
-
- 26 Oct, 2016 2 commits
-
-
Jaydeep Patil authored
The allocation of Hi/Lo part of i64 on stack has been corrected as per MIPS32 ABI. The patch also fixes ZEXT issues occurred while lowering unsigned operations. Following tests from cross-test framework were testing successfully: (non-vector, ASM mode, Om1, O2) mem_intrin TotalTests=114300 Passes=114300 Failures=0 simple_loop TotalTests=102 Passes=102 Failures=0 test_arith TotalTests=49489704 Passes=49489704 Failures=0 test_bitmanip TotalTests=1200 Passes=1200 Failures=0 test_cast TotalTests=3722 Passes=3722 Failures=0 test_fcmp TotalTests=123904 Passes=123904 Failures=0 test_global TotalTests=270 Passes=270 Failures=0 test_icmp TotalTests=3341520 Passes=3341520 Failures=0 test_strengthreduce TotalTests=240 Passes=240 Failures=0 Following tests are disabled as they are either all-vectors or contain unimplemented intrinsic lowering: test_calling_conv test_select test_stacksave test_sync_atomic test_vector_ops There are couple of fixes to ARM32 and X86 specific files occurred due to compile-time errors. R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2432373002 . Patch from Jaydeep Patil <jaydeep.patil@imgtec.com>.
-
Nicolas Capens authored
Bug swiftshader:15 Change-Id: I351268b44491091c271d6c7c5b644cd21ffb623b Reviewed-on: https://chromium-review.googlesource.com/403409Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 24 Oct, 2016 1 commit
-
-
Nicolas Capens authored
Change-Id: I70899be0455958aaad6af8d8218f1db50591beae Reviewed-on: https://chromium-review.googlesource.com/401385Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
- 21 Oct, 2016 2 commits
-
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: I3c3314f3787d42835a9483c7b797dc1dbdc0b76a Reviewed-on: https://chromium-review.googlesource.com/400663Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
Nicolas Capens authored
Change-Id: I5aee2c998842f95ccc44d5c0fed90aa289bdf67b Reviewed-on: https://chromium-review.googlesource.com/401639Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
- 20 Oct, 2016 1 commit
-
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: I7fec56518a5b4e07d2189ab01a0a223b038564c1
-
- 19 Oct, 2016 4 commits
-
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: Ic795def8a914508ab0d850c846b73b343ace45de
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: Id95a08f82c47ec20bb958358c01f389b6fb5565b
-
Nicolas Capens authored
BUG=swiftshader:9 Change-Id: Ife06416736d47acba4f2cff1ea8b17be61134752
-
Jim Stichnoth authored
src/IceTargetLoweringX86BaseImpl.h:6093:13: error: unused variable 'Src1RM' [-Werror,-Wunused-variable] auto *Src1RM = legalize(Src1, Legal_Reg | Legal_Mem); ^ src/IceTargetLoweringX86BaseImpl.h:4007:3: error: default label in switch which covers all enumeration values [-Werror,-Wcovered-switch-default] default: ^ src/IceTargetLoweringMIPS32.cpp:4065:3: error: default label in switch which covers all enumeration values [-Werror,-Wcovered-switch-default] default: ^ src/IceTargetLoweringARM32.cpp:4975:3: error: default label in switch which covers all enumeration values [-Werror,-Wcovered-switch-default] default: ^ BUG= none R=capn@chromium.org Review URL: https://codereview.chromium.org/2434643002 .
-
- 18 Oct, 2016 2 commits
-
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: I04a7c4206f3936c604ec623e43834c2a153fd3cb Reviewed-on: https://chromium-review.googlesource.com/399379Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
Sagar Thakur authored
R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2425673002 . Patch from Sagar Thakur <sagar.thakur@imgtec.com>.
-