- 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>.
-
- 17 Oct, 2016 5 commits
-
-
Nicolas Capens authored
Change-Id: I5a02aee156a64f48baca356f0a5263123f570741 Reviewed-on: https://chromium-review.googlesource.com/399590Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: I0d40fab6287130143693e8e4752859b7142a503d Reviewed-on: https://chromium-review.googlesource.com/394007Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
Nicolas Capens authored
Both vector arguments were being used in a punpckl instruction, while the shuffle mask repeats elements from just the first vector. BUG=swiftshader:15 Change-Id: I8e29c252ee4957692c4949e724ae67253b423e89 Reviewed-on: https://chromium-review.googlesource.com/399419Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Jaydeep Patil authored
The patch scalarizes Fcmp, ICmp, Cast and Select for operands of vector type. R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2412053002 . Patch from Jaydeep Patil <jaydeep.patil@imgtec.com>.
-
Jaydeep Patil authored
The patch generates HI/LO modifiers for relocatable constants. R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2420033002 . Patch from Jaydeep Patil <jaydeep.patil@imgtec.com>.
-
- 16 Oct, 2016 2 commits
-
-
Sagar Thakur authored
R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2417233002 . Patch from Sagar Thakur <sagar.thakur@imgtec.com>.
-
Jaydeep Patil authored
This patch legalizes load, store instructions post lowering. R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2411193003 . Patch from Jaydeep Patil <jaydeep.patil@imgtec.com>.
-
- 12 Oct, 2016 1 commit
-
-
Sagar Thakur authored
R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2404803002 . Patch from Sagar Thakur <sagar.thakur@imgtec.com>.
-
- 07 Oct, 2016 4 commits
-
-
Nicolas Capens authored
BUG=swiftshader:7 Change-Id: I83e51a3256365700dbaf550ed4b50c2352612f7d Reviewed-on: https://chromium-review.googlesource.com/394887Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: I4b7b97f3de18c201a502d0bc38a2c845a1caf278 Reviewed-on: https://chromium-review.googlesource.com/392627Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Jim Stichnoth <stichnot@chromium.org>
-
Nicolas Capens authored
BUG=swiftshader:15 Change-Id: Id0d3bed46d00336fc31501c41a26ebe2d4ddd697 Reviewed-on: https://chromium-review.googlesource.com/392626Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
This enables emulating 64-bit and 32-bit vectors using 128-bit vectors internally (x86 only for now). Note that these Intrinsics are not part of the PNaCL specification. BUG=swiftshader:15 Change-Id: I61a666243832c2856e60eb477d42a72dec07d01d Reviewed-on: https://chromium-review.googlesource.com/392246Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 06 Oct, 2016 1 commit
-
-
Jim Stichnoth authored
https://codereview.chromium.org/2384983002/ apparently didn't test the full "make presubmit", otherwise this would have been caught. BUG= none TBR=jpp@chromium.org Review URL: https://codereview.chromium.org/2399873003 .
-
- 05 Oct, 2016 1 commit
-
-
Stefan Maksimovic authored
This patch implements changes needed for conditional mov instructions to fix problem with failing crosstest and invalid register allocation. Problem is visible from icmp test examples, causing cross test for icmp to fail. Eg: Incorrect, before this change: 674: 00653026 xor a2,v1,a1 678: 00a3182b sltu v1,a1,v1 67c: 0082102b sltu v0,a0,v0 680: 0043180a movz v1,v0,v0 Correct, aftrer this change: 674: 00653026 xor a2,v1,a1 678: 00a3182b sltu v1,a1,v1 67c: 0082102b sltu v0,a0,v0 680: 0046180a movz v1,v0,a2 R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2394773004 . Patch from Stefan Maksimovic <makdstefan@gmail.com>.
-
- 03 Oct, 2016 2 commits
-
-
Jim Stichnoth authored
ARM support is complete, so clean up some of the lit tests: 1. Remove --skip-unimplemented 2. Use --filetype=obj instead of =asm, and remove --assemble 3. Remove --need=allow_dump requirement 4. Remove related TODOs. 5. Fix some CHECK lines because objdump output is slightly different from filetype=asm output. BUG= none R=jpp@chromium.org Review URL: https://codereview.chromium.org/2384983002 .
-
Jaydeep Patil authored
This patch implements vector operations on MIPS32 using VariableVecOn32 method (on the lines of Variable64On32). Vector operations are scalarized prior to lowering. Each vector variable is split into 4 containers to hold a variable of vector type. For MIPS32, four GP/FP registers are used to hold a vector variable. Arguments are passed in GP registers irrespective of the type of the vector variable. Lit test vector-mips.ll has been added to test this implementation. R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2380023002 . Patch from Jaydeep Patil <jaydeep.patil@imgtec.com>.
-
- 29 Sep, 2016 3 commits
-
-
Sagar Thakur authored
R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2384433002 . Patch from Sagar Thakur <sagar.thakur@imgtec.com>.
-
Sagar Thakur authored
R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2376233004 . Patch from Sagar Thakur <sagar.thakur@imgtec.com>.
-
Nicolas Capens authored
Address size prefixes are used in 64-bit x86 for PNaCl's use of the x32 ABI with ILP32 data model. Don't emit them for any other ABI. BUG=swiftshader:9 Change-Id: I1351db086d44ce4b144b3428866a54e84637b9a4 Reviewed-on: https://chromium-review.googlesource.com/390409Reviewed-by:
Jim Stichnoth <stichnot@chromium.org> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 28 Sep, 2016 4 commits
-
-
Srdjan Obucina authored
Implements SRAV instruction encoding R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2375923002 . Patch from Srdjan Obucina <Srdjan.Obucina@imgtec.com>.
-
Sagar Thakur authored
R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2369323002 . Patch from Sagar Thakur <sagar.thakur@imgtec.com>.
-
Srdjan Obucina authored
Implements MOVZ instruction encoding R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2377783002 . Patch from Srdjan Obucina <Srdjan.Obucina@imgtec.com>.
-
Srdjan Obucina authored
Implements DIVU instruction encoding R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2377733002 . Patch from Srdjan Obucina <Srdjan.Obucina@imgtec.com>.
-
- 27 Sep, 2016 1 commit
-
-
Srdjan Obucina authored
Implements intrinsic call llvm.bswap for i16, i32 and i64 R=stichnot@chromium.org Review URL: https://codereview.chromium.org/2368343003 . Patch from Srdjan Obucina <Srdjan.Obucina@imgtec.com>.
-