- 03 May, 2019 6 commits
-
-
Chris Forbes authored
Test: dEQP-VK.*d16_unorm* Bug: b/131246807 Change-Id: Ifaec74c1a2273af4585dd2423e87db4992cbf14e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30531Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com> Presubmit-Ready: Chris Forbes <chrisforbes@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Chris Forbes authored
Bug: b/131171141 Test: dEQP-VK.*a2b10g10r10* Change-Id: Ibda1017aefad8d2c045b1c6f87aa2fe2a853e313 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30530Tested-by:
-
Chris Forbes authored
Bug: b/131896622 Test: dEQP-VK.*a2b10g10r10* Change-Id: I0c50b14b007bd86dfaff39cf04938860408157f8 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30529Tested-by:
-
Chris Forbes authored
This was stomping all over memory beyond the image. Bug: b/nnn Test: dEQP-VK.pipeline.render_to_image.core.* Change-Id: Ie8738da787ea98c0d6431970cf6301bd6b5b5323 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30528Tested-by:
Alexis Hétu <sugoi@google.com>
-
SwiftShader Regression Bot authored
Change-Id: I89bf29a4dcbf978d498bbb26d5b735cb13f2ac4e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30456Reviewed-by:
-
Nicolas Capens authored
The third coordinate of the 2D sampling code is only used for 2D array textures, so we don't compute a filter factor. This used to be taken advantage of by passing the filter factor of the second coordinate to the addressing method for the third coordinate, and ensuring that the addressing mode of this component was ADDRESSING_LAYER even for non- array textures. Instead of relying on this brittle hack, just specify a third filter factor. Now that we elide materialization of single-block variables, this doesn't take any stack space and is zero overhead to the backend. Bug: b/129523279 Change-Id: Iee6b0f90bf1cc4ccc41672d3bdb706ff5af0a555 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30548 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Tested-by:
-
- 02 May, 2019 21 commits
-
-
Chris Forbes authored
Test: dEQP-VK.*b10g11r11* Bug: b/131246807 Change-Id: I3357cb5d6fccd37c1bf612fbc51380b490ded7d0 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30508Tested-by:
-
Chris Forbes authored
Bug: b/131246807 Test: dEQP-VK.*d32_sfloat* Change-Id: I1ec0d8eefa7d46422e9b6dd85766f55ae3f21179 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30488Tested-by:
-
Nicolas Capens authored
Bug: b/129523279 Test: dEQP-VK.texture.* Test: dEQP-VK.glsl.texture_functions.* Change-Id: I12991a2538edec596eedc60d5a94cc3cabfc901b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30432 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
-
Chris Forbes authored
Bug: b/131246807 Test: dEQP-VK.*e5b9g9r9* Change-Id: Ieb62847c5d86b57a81e718aa325e92e10c961c71 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30434Tested-by:
-
Chris Forbes authored
Change-Id: I8e68ecda72c9ac077c81e7a03eb53ec3c43e7812 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30433Tested-by:
-
Hernan Liatis authored
Bug: b/122837237 Change-Id: Ic2643d288dd8d217d68cda44ec291c0ea810d4d8 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30369Reviewed-by:
Hernan Liatis <hliatis@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Chris Forbes authored
These are no longer used. Change-Id: I0fde85e2b200663eac2bb32c34f90e78d76ad287 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30469Tested-by:
-
Chris Forbes authored
Bug: b/131246807 Test: dEQP-VK.*a1r5g5b5* Change-Id: Iea73786b10dca6f4116a0093218e182a089c5bcf Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30468Tested-by:
-
Antonio Maiorano authored
CMakeLists: add DEBUGGER_WAIT_DIALOG compiler definition to Debug builds of libEGL and libvk_swiftshader This matches what's in the manually managed vcxproj for the two modules. We add these for non-MSVC as well so we might eventually add support to those platforms. Note that these are guarded with #ifdef _WIN32 already, so there's no danger of this breaking non-MSVC builds. Change-Id: Ie855eb4aa1006607fdf8799ccde72305c53f3508 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30028 Presubmit-Ready: Antonio Maiorano <amaiorano@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Antonio Maiorano <amaiorano@google.com>
-
Nicolas Capens authored
Factor the duplicate code out of 2D/3D/Cube LOD calculation and add the mipLodBias field from the sampler descriptor. Bug: b/129523279 Test: dEQP-VK.pipeline.sampler.* Change-Id: I7e25fad13fbe3307dddb2dcb853ed638344ee4aa Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30429 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
-
Nicolas Capens authored
Bug: b/129523279 Change-Id: If4dfa5900ac73643cc8a8052867434d62fa1aa1e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30430 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
-
Nicolas Capens authored
Bug: b/129523279 Change-Id: Ifdb1409cf7339ac16b6fe890bc014a2e43801c76 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30431 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
-
SwiftShader Regression Bot authored
Change-Id: I3b75909a56442b2f7da626c40251ed175dd15936 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30448Reviewed-by:
-
Nicolas Capens authored
Bug: b/129523279 Test: dEQP-VK.glsl.texture_functions.* Test: dEQP-VK.spirv_assembly.instruction.graphics.image_sampler.* Change-Id: I26959eb7665af9f35dba81f83b0e0f13a4798865 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30408 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Tested-by:
-
Nicolas Capens authored
Bug: b/129523279 Test: dEQP-VK.glsl.texture_functions.* Test: dEQP-VK.spirv_assembly.instruction.graphics.image_sampler.* Change-Id: I4c02a36b35a25772e233346266ff6177a9186b9e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30249 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
-
Nicolas Capens authored
Previously only the texture pointer was passed to the sampling routine, containing some sampler data which is (still) wrongly assumed to come from a combined image-view and sampler descriptor. This change will allow splitting them up correctly. Also refactor getImageSampler to not start the Reactor function generation, moving it entirely into emitSamplerFunction so that the latter only has to be passed the pointers to the descriptor state that will affect the code that's generated. Name all the parameters according to their actual type. Clamp sampler LOD on construction so we don't have to deal with it any more later. Bug: b/129523279 Test: dEQP-VK.glsl.texture_functions.* Test: dEQP-VK.spirv_assembly.instruction.graphics.image_sampler.* Change-Id: Id3a9f12f379cf65741198b732fe387ec6e24dd86 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30350 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
-
Chris Forbes authored
Bug: b/131246807 Test: dEQP-VK.*b4g4r4a4_unorm* Change-Id: Ifb319c50cf0a9f9e83c0c8074f5631f4726cc380 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30428Tested-by:
-
Chris Forbes authored
Bug: b/131246807 Test: dEQP-VK.*a2b10g10r10_unorm* Change-Id: If51d38ddc8cbb334970e2719cb0fd96a9b240b09 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30388Reviewed-by:
-
Hernan Liatis authored
Bug: b/122837060 Change-Id: I8524a46d79e1327d494acc5ee494bad87bc9676b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30190 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
-
Hernan Liatis authored
Bug: b/122837237 Change-Id: Ic9d306e11ff5dd64f6ba0f0f6cb3a1e9a882411a Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30189 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
-
Hernan Liatis authored
Bug: b/122837488 Change-Id: I1d4b33943ea68db3c068b604876745ba6fb5097c Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30188 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
-
- 01 May, 2019 13 commits
-
-
Chris Forbes authored
sRGB decode is controlled by format, not extra state. There is also no 565 sRGB format in Vulkan. Also remove the 5 and 6bit sRGB LUTs since they are no longer used. Change-Id: Iea8770a50e4ef77004dc66d4a9867df62e7c2aaf Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30368 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
-
Chris Forbes authored
Test: dEQP-VK.*a2b10g10r10_u* Change-Id: I56498465c3cd43d23e691509c997e5c29f4fdaf9 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30348Tested-by:
-
Chris Forbes authored
- Fold together the shader -> oDepth handling. If these conditions were not the same, the shader is invalid. Change-Id: I9bec710886c9961f4141006d2d48f3eb76a3dfee Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29988 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
-
Ben Clayton authored
The only elect function we'll need to implement for Pastel 1.0. Used by memory barrier tests, but not tested on its own. Bug: b/131667233 Change-Id: I5309fa6b49c49091226fbaaca4e46363f6dff534 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30208 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Presubmit-Ready: Ben Clayton <bclayton@google.com> Reviewed-by:
-
Chris Forbes authored
The shader is run multiple times reusing the same routine state. We need to set the mask correctly before each execution. Test: dEQP-VK.glsl.discard.* Change-Id: Ic34385715cc4a16d931a3ff4fae4eeaf48d52223 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30328Tested-by:
-
Ben Clayton authored
Bug: b/126330097 Tests: dEQP-VK.glsl.discard.* Change-Id: I17632f68705aa8f0c8c607b5ec469272a58047db Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29908Reviewed-by:
-
Ben Clayton authored
Use the new UNSUPPORTED() macro where possible. Switch to using UNREACHABLE() over UNIMPLEMENTED() for switch cases that do not exist according to the spec, or for cases that should be handled by validation in an earlier stage. Bug: b/131243109 Change-Id: Id9970b64cf248468c508e65366bc576cb03e8e58 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29929Reviewed-by:
-
Ben Clayton authored
Prevents typos that can cause undefined behavior. Bug: b/127433389 Change-Id: I6308831d9ae8ff3d6147c5a766de81fdc239fab5 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/29931Tested-by:
-
Nicolas Capens authored
Also fix Lod + Offset combination, and rename the parameter that holds either lod or bias to lodOrBias. Bug: b/129523279 Test: dEQP-VK.glsl.texture_functions.* Change-Id: I397959421651e990d041f4297c4bcf55e1c00f03 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30229Reviewed-by:
-
Chris Forbes authored
TODO: figure out what is happening with linear filtering. Possibly pre-existing breakage -- if we enable filtering support for R32_SFLOAT that fails in the same way. Test: dEQP-VK.texture.* Test: dEQP-VK.image.* Test: dEQP-VK.pipeline.* Change-Id: Ia461418d772eb5aceb101b84eaa239b0c0bce2c0 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30288Tested-by:
-
Ben Clayton authored
Tests: dEQP-VK.spirv_assembly.instruction.compute.memory_access.* Change-Id: I5281798468651c67ec038bdd8688f99c67654cb1 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30215Tested-by:
-
Ben Clayton authored
Fixes cases where functions were not being inlined. Tests: dEQP-VK.glsl.functions.control_flow.* Change-Id: Ia752517040f7529ec3b47adc75ec03646d183f6d Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30214Tested-by:
-
Ben Clayton authored
Non-constant initializers not yet implemented. Tests: dEQP-VK.spirv_assembly.instruction.graphics.variable_init.* Tests: dEQP-VK.spirv_assembly.instruction.compute.variable_init.* Tests: dEQP-VK.spirv_assembly.instruction.compute.opunreachable.* Tests: dEQP-VK.spirv_assembly.instruction.compute.shader_default_output.int.initialized Bug: b/131681817 Change-Id: I8b283042d3be15028c2efe10988d6847c6256811 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/30213Reviewed-by:
-
