- 12 Apr, 2019 8 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>
-
SwiftShader Regression Bot authored
Change-Id: Icf1d6ad270eeeb71ce2601f01dca0248043459cf Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28928Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
- 11 Apr, 2019 32 commits
-
-
Ben Clayton authored
This is done so that we can build for both the subzero and llvm backends. Old configs remain where they are during a transition period. Bug: b/130343040 Change-Id: Icc9e1b92074ef90aacfd9e6d40b021d35f4d9ef3 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28908Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/130335507 Change-Id: I4cd5ab717bab39f1b03a4c6e8140603aa36c2fff Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28869Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Bug: b/123933266 Change-Id: Ia7b59b1c6edbc2180ed7ff39ecce4bfdca20804a Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28828Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Ctlz and Cttz are stubbed for now. Will fix after the builds turn green. Bug: b/130343040 Tests: dEQP-VK.glsl.builtin.precision.* Tests: dEQP-VK.glsl.builtin.function.integer.findMSB.* Change-Id: Ibff6a52bee7f5440c536dfc7a61ad4ba3e96743d Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28868Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Change-Id: I94a2fd3a900fb1e2c625eaa76dcce8fa785360cd Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28850 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Alexis Hetu authored
There were a bunch of warnings on Windows: - Precision loss through type conversion - Switch statement containing only default case - Zero sized array Bug b/130335507 Change-Id: I809db29db16f5dfd62d03d40353f4f2e0f6c3c93 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28768Tested-by:
Alexis Hétu <sugoi@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Alexis Hetu authored
Added functions to return properties for external objects. All properties are currently disabled. Bug b/117974925 Change-Id: I3322ab7125bddef41f50b0160c22687a1182ae16 Tests: dEQP-VK.api.external.* Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28770Tested-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> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
SwiftShader Regression Bot authored
Change-Id: Ib23900cfd0c7d7013301dbee559be5a42256ccf5 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28829Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Nicolas Capens authored
Should speed up the build a little. Also removes Minimal Rebuild. It has been deprecated: https://docs.microsoft.com/en-us/cpp/build/reference/gm-enable-minimal-rebuild?view=vs-2019 Bug b/129772660 Change-Id: I44622edc2fc88ca6b2b6bcb6a515d1e3326d3dbd Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28769 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Stub all functions unimplemented by the subzero backend. Change-Id: I9dd3a1f7dd633934f4ee76d6807a89a4b117232a Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28849 Presubmit-Ready: Ben Clayton <bclayton@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Remove duplicate definitions of Extract and Insert for UInt4 Change-Id: Ib2d54606ead6b953b93f28e9f0dfd7987b1da774 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28848 Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
... that will not be required for Pastel 1.0. Bug: b/126873455 Change-Id: I024296dd39ae74613577bd5dbd8d75051b7f4f5c Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28790Tested-by:
Ben Clayton <bclayton@google.com> Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.function.integer.findlsb.* Change-Id: I46671fe6b64814a5c9cbc8dd9fe4cc449a328f42 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28789Tested-by:
Ben Clayton <bclayton@google.com> Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.function.integer.findMSB.* Change-Id: I6337998f725334c31d8adeae05fc441bb3895908 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28788Tested-by:
Ben Clayton <bclayton@google.com> Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.matrix.determinant.* Change-Id: I81ae56951b9fa2e1aedb327039160cf44f7f0a18 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28728Tested-by:
Ben Clayton <bclayton@google.com> Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Beware: the dEQP tests here are very weak - they do not seem to properly test the NaN handling of this instruction. Bug: b/126873455 Tests: dEQP-VK.spirv_assembly.instruction.compute.opnclamp.all Change-Id: I8d1f80f5536996ca46c30ee4895d9f6e2c29a03e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28711Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Beware: the dEQP tests here are very weak - they do not seem to properly test the NaN handling of this instruction. Bug: b/126873455 Tests: dEQP-VK.spirv_assembly.instruction.compute.opnmax.all Change-Id: Ibd704c9dc9c3475df10b02c0c0f80b9c3e472ff4 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28710Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Beware: the dEQP tests here are very weak - they do not seem to properly test the NaN handling of this instruction. Bug: b/126873455 Tests: dEQP-VK.spirv_assembly.instruction.compute.opnmin.all Change-Id: I5283af4a1e697b080f53193bc21ddd446b232d1e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28709Reviewed-by:
Chris Forbes <chrisforbes@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
Was removed from the spec in version 0.99. Bug: b/126873455 Change-Id: I93c60b7d1e8332e70f7116636384f4f85ea542cb Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28708Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
RcpSqrt_pp is not accurate enough to be used on non-relaxed precision types. Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.inversesqrt.* Change-Id: I3e34d28dbd863c9cb14cf6c79afe08a1992b5478 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28697Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
While not strictly necessary (we could always calculate with the highest precision), it is a tiny amount of work to dramatically increase performance of instructions like InverseSqrt for relaxed precision types. Bug: b/126873455 Change-Id: I42d5871a8e0886001e85fd4e16f8e6cb309236cf Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28696Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.sqrt.* Change-Id: Ia178618bb2777d9b58beef19d4e18767cc9a8f73 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28695Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.log2.* Change-Id: I1a6a4e1fbf41ccd7aacf2fa7fbe8b8e7c6d28f3b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28443Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.exp2.* Change-Id: I8a8f03a9dd1e30866b6f330c987d93bf4d95d265 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28442Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.log.* Change-Id: I14cc590230dbbf4cfd24a0903bc2fbfb3c5a5192 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28441Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.exp.* Change-Id: I35a28475b56d820b6306782188d72a151845fe58 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28440Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.pow.* Change-Id: I74797e69adc78e4c06ec4a8b2923b78166afbc18 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28439Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.atan2.* Change-Id: I85596109069d3cee8ebd5f65770cb06ca7b93e53 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28438Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.atanh.* Change-Id: Ie58753a4a0bdef18ec317129f2c4a7f0c0c314b4 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28694Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.acosh.* Change-Id: Id02fae0c7020ee83cb612f9370cdbb5c4dc31af1 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28693Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.asinh.* Change-Id: I83b425bea2d693c5bcab3b1685252be0ba9fb023 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28692Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/126873455 Tests: dEQP-VK.glsl.builtin.precision.tanh.* Change-Id: If63f53b5d9d78dd4a2cd6c31c42cb16f03a48d98 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/28691Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-