- 17 Apr, 2019 1 commit
-
-
SwiftShader Regression Bot authored
Change-Id: Ica7f55af1f6ddeea81fb1560ec7d58d314de9c07 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29388Reviewed-by:
Chris Forbes <chrisforbes@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
- 16 Apr, 2019 8 commits
-
-
Chris Forbes authored
- BuiltInSubgroupSize - BuiltInSubgroupLocalInvocationId Bug: b/118386749 Test: dEQP-VK.subgroups.* Change-Id: Idf06213351f2ac27664cc6bee1b6b99575676843 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29308 Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Chris Forbes authored
Bug: b/118386749 Test: dEQP-VK.renderpass.* Test: dEQP-VK.glsl.* Change-Id: I1f830c3fbe5b1f6bca0a6f17dd37fffd00e0cfa3 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29249Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Chris Forbes authored
Bug: b/118386749 Test: dEQP-VK.renderpass.* Test: dEQP-VK.glsl.* Change-Id: If3d603f860a046c06c4bbc0390206f0720ff6c82 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29248Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Chris Forbes authored
- On LE machines, A8B8G8R8_*_PACK32 are the same as R8G8B8A8 - Add support for B8G8R8A8_SRGB in various places Bug: b/118386749 Test: dEQP-VK.renderpass.* Change-Id: I16d587b72317a05277fe2c8b337af5670eb51310 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29228Tested-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
C++ 11 is in heavy use by SwiftShader now, so SwiftShader is now unsupported for Android versions earlier than M. Bug: b/130460252 Bug: b/130343040 Bug: b/130361288 Change-Id: Idfb64c3fa82325641cc2170f494260c1b5b1eeb4 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28988Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: Lingfeng Yang <lfy@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
SwiftShader Regression Bot authored
Change-Id: Ic300120b552abc353e2a3866f8e6d3381d3d5e9a Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29288Reviewed-by:
Chris Forbes <chrisforbes@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Chris Forbes authored
- Remove Primitive::area, it is unused - Always set front/back masks - Initialize frontfacing based on front/back mask Bug: b/118386749 Test: dEQP-VK.glsl.builtin_var.* Change-Id: I8ccb5a2deb64df2dc4774a3511a5d30c09c30e5f Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29208 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Chris Forbes authored
Bug: b/118386749 Test: dEQP-VK.draw.* Change-Id: I6064e47825f1dcc2a910fe4be933868a4f3835b5 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29169Tested-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>
-
- 15 Apr, 2019 11 commits
-
-
Ben Clayton authored
Fix up all calls to `assert()` in [LLVM,Subzero]Reactor.cpp with an appropriate call to one of these macros. Bug: b/127433389 Change-Id: I188add3929c46932b8de5acf2ac4b2ac83b0768b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29055 Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Chris Forbes authored
We claim no support for now. Bug: b/117974925 Test: dEQP-VK.draw.shader_draw_parameters.* Change-Id: Id3b09419c365cc513dabf636a932c087b99f7cc3 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29188 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
Bug: b/118386749 Test: dEQP-VK.pipeline.* Change-Id: Ie549802e6ea9ad89760e2439de5776121fae8c58 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29028Tested-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com>
-
Chris Forbes authored
These functions are part of the VK_KHR_swapchain + VK1.1 interaction. Test: dEQP-VK.wsi.xlib.* Bug: b/124265819 Change-Id: Ic79bb0c290e68ab73ddc589c95f979603ecd1a82 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29168 Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Alexis Hetu authored
Intentional allocation failure tests were crashing on Linux, because the object's constructor was called even when the allocator returned nullptr. Separated the allocation from the construction to fix the issue. Bug b/116336664 Change-Id: I7a5d4e957ec27f37a96b795a7f17aacebb240fe9 Tests: dEQP-VK.api.device_init.create_instance_device_intentional_alloc_fail Tests: dEQP-VK.api.object_management.alloc_callback_fail.* Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28948Tested-by:
Alexis Hétu <sugoi@google.com> Presubmit-Ready: Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
SwiftShader Regression Bot authored
Change-Id: I1078943fbb5e030845563b1f102bf461cc8fa94b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28930Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Alexis Hetu authored
Pipeline variables, like scissor, viewport and blend constants, may not be provided by the pipeline state if they are set to use the dynamic state instead. In that case, the related initialization structures may be null and should not be used. Bug b/118619338 Change-Id: Ieb5ff066b06ecfe444c6d26e43c70fd18d2f9c31 Tests: dEQP-VK.image.texel_view_compatible.graphic.extended.* Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29053Tested-by:
Alexis Hétu <sugoi@google.com> Presubmit-Ready: Alexis Hétu <sugoi@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Alexis Hetu authored
Fixed ImageView::imageTypesMatch() according to the Vulkan Spec, Section 11.5. Image Views, Table 15. Image and image view parameter compatibility requirements. Also allowed already supported mip level parameters to be used in the clear function. Bug b/119620767 Tests: dEQP-VK.pipeline.render_to_image.core.cube.* Change-Id: I4785c4594ba82f8e97b40aaddce8c950d255c846 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28748 Presubmit-Ready: Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Alexis Hétu <sugoi@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Alexis Hetu authored
Implemented the resolve command using the regular blit function, since it already supports resolve operations. Removed baseArrayLayer related checks in Image::copyTo(), since resolving array images exercises that path. Bug b/118619338 Change-Id: I6a70ef5f396e51be7fde34ebe72fcf991396a45f Tests: dEQP-VK.api.copy_and_blit.core.resolve_image.* Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28888 Presubmit-Ready: Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Alexis Hétu <sugoi@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
This now builds both the LLVM and Subzero Reactor backends. Fixed issue where failures were not actually failing the build in continuous.bat. Changed the test's expected path of vk_swiftshader.dll to more closely match macOS and Linux. Bug: b/130458775 Bug: b/123360006 Change-Id: Id2cfc0a7016c9813597567fd6ab900fe4047b8f8 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29109 Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Makes reading Windows Kokoro build logs possible on bad WiFi. Bug: b/123933266 Bug: b/115344057 Change-Id: I4f336f8bd149e99f9926853fc09b79f7a063ea4b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29148Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
- 13 Apr, 2019 13 commits
-
-
Ben Clayton authored
This reverts commit a786c4a2. Reason for revert: Breaks LLVM 3 reactor backend. Change-Id: Ia7353182bbeab8f357bd9e4dababcc24e6cdd811 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29128Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
This reverts commit 0179e5eb. Reason for revert: Breaks LLVM 3 reactor backend. Change-Id: I0cdeac496c25b8b8994339e0b4b5a002d2e597f5 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29129Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
Moved the hand-rolled implementation to Subzero. As we've started exposing bit intrinsics, we might as fix the TODOs. Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.function.integer.bitcount.* Change-Id: Ic37dfd5d73187f2b3afa444abfd9e22439c871b1 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28792 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
Moved the hand-rolled implementation to Subzero. As we've started exposing bit intrinsics, we might as fix the TODOs. Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.function.integer.bitfieldreverse.* Change-Id: Ifadf25045b9ee4b435d9f6750c63334b7d4cad83 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28791 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>
-
Alexis Hetu authored
Implemented all dynamic state commands and applied the proper state within the draw command. Bug b/118619338 Change-Id: Ifeca42be1698f642e137e807aa59958447921fcc Tests: dEQP-VK.dynamic_state.* Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28890Tested-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
Change-Id: I83a615291e32e3d08e63a84a71ee1de7641c5bef Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29068 Presubmit-Ready: Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Most of these come from LLVM 3. I'd normally try hard to silence these warnings without modifying 3rd party code, but I indend to delete this dependency in the very near future. Bug: b/130343040 Change-Id: Icf6003c58ad118989240d64cf480aa1e2e639fb7 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29088 Presubmit-Ready: Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
SwiftShader Regression Bot authored
Change-Id: I751f9ffdf788c16bc7dc33bffc15df519c705b2f Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28929Reviewed-by:
Chris Forbes <chrisforbes@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Ben Clayton authored
Bug: b/130343040 Change-Id: I413e35e8df6d3609399b33f6696661754e205d24 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29032 Presubmit-Ready: Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com>
-
Ben Clayton authored
These currently do not work. Bug: b/130343040 Change-Id: Ia97b611f44a5a048f324daa236e84e933a2c6b67 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29033 Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
Will be fleshed out in another set of changes. Bug: b/130343040 Change-Id: I3df91cbeb60aba555020e21ffdf6feaa12865389 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29031 Presubmit-Ready: Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/130343040 Change-Id: I4e69330929ddc81f840c01fe9ceb8a06d5869f53 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29049 Presubmit-Ready: Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Promote this up from the LLVM specific build settings. Bug: b/130343040 Change-Id: I0d916b2635c79d3f292abf659973c570be05fdd8 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29029 Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
- 12 Apr, 2019 7 commits
-
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.matrix.inverse.* Change-Id: Ie4cbe7a193ac18e383cc2c783df1eca21d0dcbf1 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28808 Presubmit-Ready: Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Replaces VkDescriptorImageInfo with a more concrete SampledImageDescriptor, which contains an sw::Texture that SamplerCore uses. The parameters can be looked up statically only because the shaderSampledImageArrayDynamicIndexing feature is currently not supported. Bug b/129523279 Test: dEQP-VK.texture.filtering.2d.formats.r8g8b8a8_unorm.nearest Change-Id: I619b5b48b2b4552d9bfc70b087df2c31eabb49ea Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28434 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com>
-
Chris Forbes authored
Bug: b/128690261 Test: dEQP-VK.glsl.* Test: dEQP-VK.ubo.* Test: dEQP-VK.ssbo.* Change-Id: Ibafc5f64263cd627a3a2cd961af226cb5b110ea0 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28968Tested-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>
-
Chris Forbes authored
Bug: b/128690261 Change-Id: I847ac2d1727066076c0eae56cfdd94bf31c1f400 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28949Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com>
-
Chris Forbes authored
[Note: New change-id as I'm experimenting with moving this on top of Nicolas' decoration tweaks] Bug: b/128690261 Test: dEQP-VK.glsl.* Test: dEQP-VK.ubo.* Test: dEQP-VK.ssbo.* Change-Id: I535c07964cd74dd4fd9c92c179aa34c53b3ff907 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28910Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Nicolas Capens authored
The at draw time currently bound descriptors are passed down to the SPIR-V code generation so that we can read the sampler parameters and image view parameters. An important exception is compute shaders, which are currently compiled before the invoke calls. Also still TODO is taking the parameters into account during routine lookup in the cache. This change also eliminates the now unused 'enableMask' from PixelProgram and VertexProgram. Bug b/129523279 Change-Id: I1ea75ec5a7b83783a1efacb238143a419afa5bda Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28446Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Nicolas Capens authored
Split descriptor decorations from other decorations since they're always used separately and this makes propagating them more lightweight. Bug b/129523279 Change-Id: I2f5acecc990bf15c7feb4ce81539bd2b5818bdf6 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28445 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Nicolas Capens <nicolas.capens@gmail.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com>
-