- 13 Mar, 2019 7 commits
-
-
Dan Sinclair authored
When embedding SwitftShader into a third party application, the top level cmake files may already include googletest and SPIRV-Tools. This CL updates the SwiftShader tests to only try to pull in googletest if the `gtest` target is not found and only pull in SPIRV-Tools if the `SPIRV-Tools` target is not found. Change-Id: I3c3ebb528725017dbbb80aa05b01ee2de8d4a2d1 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26788Reviewed-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Dan Sinclair <dsinclair@google.com>
-
Alexis Hetu authored
Removed sw::Surface dependency from: - Blitter - Context - PixelProcessor - PixelRoutine - Renderer - VkFramebuffer No behavior changes should be happen with this cl. Bug b/126883332 Change-Id: I4146f043ba8e5008eb3d44465c26a0f34a2e1acc Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26969Tested-by:
Alexis Hétu <sugoi@google.com> Presubmit-Ready: Alexis Hétu <sugoi@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com>
-
Ben Clayton authored
Change-Id: I1881916302d6652144138106d0744e9bbd2b45c8 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/27008 Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
SwiftShader Regression Bot authored
Change-Id: I5d87130b1c52cd7c3e2a21d1a1f0fcbe6d2b40ed Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/27049Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Chris Forbes authored
D16_UNORM depth operations are performed in fixed point to avoid quirks of ushort <-> float roundtrip Some slight oddities to work around missing primitives: - We only have CmpEQ, CmpGT. This matches SSE hardware reality. Everything else is made out of negations and operand swaps. - We only have *signed* CmpGT. Make an unsigned CmpGT by biasing both 16-bit operands by -0x8000. - In non-quad-layout we don't have a good way to separate .xy and .zw halfs; reading via masked short4 reads, writing via individual component writes. Bug: b/128363727 Test: dEQP-VK.pipeline.depth.format.* Change-Id: I9c8b249470a9c91589c91135988dc4d1a58bbc5f Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/27030Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Chris Forbes authored
Bug: b/128363727 Change-Id: Ia595876610b077c4be366174264de7a55f14dc4c Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/27029Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Chris Forbes authored
Also add implementations of multiplication and right shift for Long type. Bug b/126873455 Change-Id: I9952c2b9a3feca6a7741cd02e2295340935e4447 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/25988Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com>
-
- 12 Mar, 2019 15 commits
-
-
Hernan Liatis authored
Bug: b/124265819 Change-Id: I7c84c3be8fb13a4ef798f90b8c610ebeafa82b5c Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26970Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Hernan Liatis <hliatis@google.com>
-
Hernan Liatis authored
When we create a new swapchain, we can pass in a parent swapchain from the same surface. The old swapchain is no longer associated with the surface and "retires" by deleting all non-acquired images. Bug: b/124265819 Change-Id: I31315045c926b929483d1b6cb1fc3d9f7aa87b94 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26668Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Hernan Liatis <hliatis@google.com>
-
Chris Forbes authored
Bug: b/128363727 Change-Id: Id75e138deda980f57b7bd07192b1353b2b414fd8 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26988Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Hernan Liatis authored
In summary, this change associates a 1:1 correspondance between a VkImage and XImage. This will fix resize issues as well as prepare to implement createInfo->oldSwapchain. Bug: b/124265819 Change-Id: I42e8e1e7b5c9a5b64820156603dca10e2ab828ca Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26368Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Hernan Liatis <hliatis@google.com>
-
Alexis Hetu authored
Removed unused variable Change-Id: I9236d02e04fb58a081c608d16f4e1eadf81dde16 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26949Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com>
-
Nicolas Capens authored
- Make VulkanUnitTests depend on SPIRV-Tools. - Build Vulkan DLLs but don't link statically with VulkanUnitTests. - Don't use Unicode character set. - Statically link runtime library. - Fix narrowing conversion from 'VkCommandBufferUsageFlagBits' to 'VkCommandBufferUsageFlags'. Bug swiftshader:125 Change-Id: I7dbdc25c829cc76b5e05cd57cd9f85bf2d4e0903 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26948Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Hernan Liatis authored
The following changes will allow to present to the screen. However, do not adjust the size of the window. Bug: b/124265819 Change-Id: Iaa42f458af9555e91d47238397112324ec080a67 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26008Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Hernan Liatis <hliatis@google.com>
-
Chris Forbes authored
I need this in order to support depth formats that arent D32F. Bug: b/128363727 Change-Id: I7016351207e433e668053ebaf4dbeb9f2f7936f2 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26888Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com>
-
Chris Forbes authored
- Many cases where sampleCounts is required to be VK_SAMPLE_COUNT_1_BIT by spec - Missing feature flags for ETC2 formats - Max array layers must be 1 for 3D See 38.1.1 Supported Sample Counts for details of sample count requirements Bug: b/128344974 Test: dEQP-VK.api.info.image_format_properties* Change-Id: I3ffe747a8cc8d06e0d33aa0937bd2b7ecd045ff5 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26871Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Alexis Hetu authored
In order to eventually remove sw::Surface, some utility functions used by vk::Image were copied to vk::Format. Bug b/126883332 Change-Id: Ie8404b70adc3336d536dcd5c0ec26b63e46c5174 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26872Tested-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
We weren't looking at the most recent change. Kinda important when that's typically the new test list you're trying to export. Change-Id: I0e312dbde0ae54feb0641a658eebc73bbdb47eaa Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26928Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
Bug: b/126871859 Change-Id: I0b3db6c033419a2ad54453d470960330d4f337cc Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/25909 Presubmit-Ready: Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
This is by no means finished: * ComputeProgram holds the codegen and the invoke logic. ComputeProgram::run() should probably be moved somewhere else (maybe Renderer?). * ComputeProgram::run() is currently fully single threaded. * The compute routines are currently not cached. With that said, this is enough to start passing a whole bunch of dEQP compute tests. Test: *.compute.* Bug: b/126871859 Change-Id: Ic6a76826b2fec0d34d54e0bed564b360ea0610c0 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26551 Presubmit-Ready: Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Change-Id: I6e695dd5221da6d9f7f198bc2af8aee00eda8881 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26808 Presubmit-Ready: Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
SwiftShader Regression Bot authored
Change-Id: I15fb0452789a7a3e3f84453a1b2d9a1b969d510f Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26908Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
- 11 Mar, 2019 9 commits
-
-
Alexis Hetu authored
The sw::Surface object was the intermediate representation of an image used between the vk::Image and the Blitter. This cl removes the need for an intermediate representation by having the Blitter use the vk::Image directly. There should be no regression for the clear and blit tests. Bug b/126883332 Change-Id: Icbc15470e3ad112ed78f4f62d6d82c66e3e37a20 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/25928Tested-by:
Alexis Hétu <sugoi@google.com> Presubmit-Ready: Alexis Hétu <sugoi@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com>
-
Ben Clayton authored
The idea here is that we can build a number of fine granularity SPIR-V tests, which we can also verify against any optional system Vulkan hardware. This CL adds support for building and running the Vulkan unit tests on mac and linux. Bug: b/123749916 Change-Id: I01a4e2a914ffd5473d271013957fa3dddbc43645 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/24328 Presubmit-Ready: Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Nicolas Capens authored
Bug b/123360006 Change-Id: I4132ebf474f8055554126529839c33fd87d75499 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26768Tested-by:
Nicolas Capens <nicolascapens@google.com> Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Nicolas Capens authored
- Add clarifying comments. - Add self-explanatory helper functions. - Fix typeSize to handle all Reactor types. - Use uintptr_t base type for emulated type enum. Prevents 0x########00000000 from being interpreted as emulated type. Bug b/126028338 Change-Id: Ib38cf9b59f58c0f6046f9c84a6152849791bb4ed Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26648Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
Test: *smod* Bug: b/126873455 Change-Id: If946e72d23aa9386b85cffe96b02ab63195f0a24 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26549Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Exports the latest test data to a Google Sheets document. Change-Id: Ia1b38464daf7117da571d536e7ff029023b9de58 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26748Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Nicolas Capens authored
This causes Visual Studio to default to the x64 configuration, which is 64-bit instead of 32-bit. Change-Id: Ifb1246216c726e6ad0dcc7891f2f3a8f89aeff7c Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26649Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Fixes tests that turned from PASS -> CRASH by adding compute shader support. Bug: b/120799499 Change-Id: I72190eef98223851cba3d61a98d54ab2f6e2a0a7 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26553Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com>
-
SwiftShader Regression Bot authored
Change-Id: If451dc09203631737739b30d385fbf8a215b2b46 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26728Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
- 09 Mar, 2019 7 commits
-
-
Ben Clayton authored
The test list from the latest patchset is now run on the parent. Missing tests are a bug. Change-Id: I149e235968acad4988d0fcff78d8a8796aaf118c Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26559Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Change-Id: Ib0beec96a6efafa0ee6fb8a43e15142206d3cd4d Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26558Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
This is better than running potentially two different test lists for both. Requires caching the parent test results with the testlist's hash as a cache key. Change-Id: Ic153e65395fc1c3f02a66bbb1df60731ce28901b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26557Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/123933266 Change-Id: I4585c881ea7328b03b7c98bd5b4c255a399a0a70 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26532Reviewed-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
SwiftShader Regression Bot authored
Change-Id: I6cd54eefee8e4fe0a20799bf6bce85ddd53df9d5 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26708Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Chris Forbes authored
This gets us the final few cases in the glsl multiplies group. Test: dEQP-VK.glsl.operator.binary_operator.mul.* Bug: b/126873455 Change-Id: Id9ec4ad1754a7e63c8d7e0aa0a0b0b156a03b024 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26651 Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Chris Forbes authored
- FMin - FMax - SMin - SMax - UMin - UMax - Step - SmoothStep - FMix - FClamp - SClamp - UClamp - FSign - SSign Test: dEQP-VK.glsl.operator.common_functions.* Bug: b/127804400 Change-Id: Ie136d8f0386f6a1db9acbf4e261851ce696e2d95 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26650Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
- 08 Mar, 2019 2 commits
-
-
Chris Forbes authored
Mostly trivial; exception is RoundEven, for which I borrowed the implementation from the GLES side. Test: dEQP-VK.glsl.operator.common_functions.* Bug: b/127804400 Change-Id: I889c3750ebdbc078723890488cf44d89ea77103b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26628Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Chris Forbes authored
Test: dEQP-VK.glsl.operator.bool_compare.* Test: dEQP-VK.glsl.operator.binary_operator.equal.* Test: dEQP-VK.glsl.operator.binary_operator.not_equal.* Bug: b/126870789 Change-Id: I9d33444d67792274d8027b7d1632235533cfc079 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26574Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-