- 15 Jan, 2020 1 commit
-
-
Ben Clayton authored
37f30185 added CMake rules to copy the `OpenCLDebugInfo100.h` header out of a polluted build intermediate directory and into a separate include directory. Add this include directory to the .vscode include paths so vscode can also find it. All of this smell should go away once https://github.com/KhronosGroup/SPIRV-Headers/issues/137 is addressed. Bug: b/145351270 Change-Id: I1a20e6f6b507e10c90fed38dd8a94d4b3ff590f7 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40129Reviewed-by:
Antonio Maiorano <amaiorano@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
- 14 Jan, 2020 3 commits
-
-
Ben Clayton authored
Instead of casting to int for a map key. Bug: b/147359661 Change-Id: Id777f7285f9349e7ab855c4a5a26d6a2aa4c02ba Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39953 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
Bug: b/147359661 Change-Id: If36058304196625465c2b632d789617e0947352a Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39952Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Chris Forbes authored
Bug: b/141698760 Change-Id: Icb293abc63a46c41a5165cdaa2061c23926b2568 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39988 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
- 13 Jan, 2020 9 commits
-
-
Ben Clayton authored
Variadic arguments weren't using the ## prefix, and causing build failures on certain compilers. Also always `printf()` the "Waiting for debugger connection..." message. If we're debugging, then writing to `stdout` isn't an offence. Bug: b/145351270 Change-Id: I2f86b118043ef2b2b6d291834c9f134a4db5c18d Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40091Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
SPIRV-Tools generates extension header files into the the project's intermediate build directory. SpirvShader needs to include `OpenCLDebugInfo100.h`, so copy this out to a "clean room" include directory and add this directory to the vk_swiftshader INCLUDE_DIRECTORIES. All of this smell should go away once https://github.com/KhronosGroup/SPIRV-Headers/issues/137 is addressed. Bug: b/145351270 Change-Id: Ib9205e04d32e2d8e0a6516449ec9114a3ff6dd94 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40090Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
Define ENABLE_VK_DEBUGGER to 1 so that VS code doesn't gray-out the debugger logic. This does not affect the build in any way. Bug: b/145351270 Change-Id: I52e1dfa7c2d83f2db4258d6e9e5f3a57c601452d Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40089 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
Includes the fix for invalid SPIR-V merge blocks. Removed the temporary patch added in e75a1838. Change-Id: Ibab6c3606077a4b7207884d24bbf1232207d1800 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40128Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Nicolas Capens authored
That's what it's called by the Vulkan spec. Fixes: b/29203875 Change-Id: Id5049147692b0c845b1bb7afeb8ff5fa94c25773 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40114 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com>
-
Nicolas Capens authored
This extension is believed to be unused. Bug: b/147536183 Change-Id: Ifaa1e560414e93f2d1382d7d83c335dda9370eee Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40110Tested-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com>
-
Nicolas Capens authored
This extension is believed to no longer be required. Note GL_EXT_texture_format_BGRA8888 offers similar (but not identical) functionality. Bug: b/147536183 Change-Id: Id58c62f5ab2fddb3f99e43a9ab74a5929d29983a Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40108 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>
-
Nicolas Capens authored
In rare cases the requirement to have a Buganizer or Monorail bug ID in the commit message can be unreasonable. For example when the build is broken and the fix is trivial so the creation of a tracker bug would needlessly take valuable time. Another example would be fixing a spelling mistake in a comment and we don't want to spam the tracker thread with a commit notification. For such cases, accept bug info lines containing no digits. Bug: b/141892461 Change-Id: Iaf44eb3dfc08d3e138fe8ba24501fa9c046d36de Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40029 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
Nicolas Capens authored
Previously any line starting with Bug: was accepted. This change restricts it to Buganizer and Monorail bugs of the form: Bug: b/141892461 Bug: swiftshader:123 Note nothing else is accepted before or after this match (including whitespace). Multiple bug references must go on separate lines. This ensures the bug ID is recognized correctly by Buganizer, Monorail, and Gerrit tools that parse them out. Bug: b/141892461 Change-Id: I910b18eba1a2bb473f2713c088c408cb31867487 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39910 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Antonio Maiorano <amaiorano@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 11 Jan, 2020 2 commits
-
-
Ben Clayton authored
Generate a synthetic file containing the spirv disassembly, and allow the debugger to single line step over these instructions, along with inspection of the SSA values. All of this is no-op unless ENABLE_VK_DEBUGGER is defined at compile time, and the VK_DEBUG_PORT env var is set at run time. Bug: b/145351270 Change-Id: Iba71717d78f7213ba586a1632b44f5fe08addf08 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38915 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Antonio Maiorano <amaiorano@google.com>
-
SwiftShader Regression Bot authored
Change-Id: I0595a5d7b1bfe20a063eb98caf7497a36e9daa2c Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40088 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Alexis Hétu <sugoi@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
- 10 Jan, 2020 5 commits
-
-
SwiftShader Regression Bot authored
Change-Id: Ibe0a658c20ccf2fba47d5d0badb0764150e7a8a9 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40048 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Chris Forbes <chrisforbes@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Alexis Hetu authored
Bug: b/144825072 Change-Id: If620d4ce920dad1d929c8e1ad3ea87d8de8f9b1e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40030Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Alexis Hetu authored
A2B10G10R10 (RGBA) was already supported by SwiftShader, but A2R10G10B10 (BGRA) was not. Most of this cl is trivial, as it simply adds an equivalent version of the new formats from the code used for the already supported formats, with the R and B channels swapped. The only new piece of code is rounding for 1010102 formats at the top of the PixelRoutine::writeColor() function. There was already rounding for 8 bit formats, but not 1010102, which led to potential off by 1 errors in the output, which is fairly large when it happens on the 2 bit alpha channel. This fixes one of the dEQP-VK.pipeline.blend.*a2r10g10b10* tests. Tests: dEQP-VK.*a2r10g10b10* Bug b/142661203 Change-Id: Ifcae17aecafab3ea7967fdc755391ddd5e651ca5 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40008Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Ian Elliott authored
The logic of PhysicalDevice::getImageFormatProperties(), as well as testing with ANGLE, shows that multisampling of integer formats is supported and works. The value of VkPhysicalDeviceLimits::sampledImageIntegerSampleCounts should therefore be set to: VK_SAMPLE_COUNT_1_BIT | VK_SAMPLE_COUNT_4_BIT Bug: b/147438583 Change-Id: Iaa7bd22789fd20b4e7975d3d2ffee2a8f90b006b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40068Reviewed-by:Nicolas Capens <nicolascapens@google.com> Tested-by:
Ian Elliott <ianelliott@google.com>
-
Sean Risser authored
The primary purpose of doing this is to have all the PowerVR SDK examples in one place and remain up-to-date. This change also lets us simplify a bit of our CMakeLists.txt as we only need to use add_subdirectory to build everything we need. The PowerVR SDK examples are built by explicitly enabling SWIFTSHADER_BUILD_PVR_EXAMPLES (either through a cmake gui, or -DSWIFTSHADER_BUILD_PVR_EXAMPLES=ON on the command line). You must also specify a window system for the examples to use with something like "-DPVR_WINDOW_SYSTEM=X11". Several cmake option flags were renamed to start with "SWIFTSHADER_" to avoid name collisions with subdirectories. Bug: b/141361933 Change-Id: I6d2fac579f62ff4bc211d5fd5127fb670eb9b4bf Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39909Tested-by:
Sean Risser <srisser@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
- 09 Jan, 2020 5 commits
-
-
Alexis Hetu authored
This is a follow-up to this cl: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39748 where filtering was enabled for depth only formats. According to the Vulkan spec (about VK_FORMAT_FEATURE_SAMPLED_IMAGE_FILTER_LINEAR_BIT): "If the format being queried is a depth/stencil format, this bit only specifies that the depth aspect (not the stencil aspect) of an image of this format supports linear filtering, and that linear filtering of the depth aspect is supported whether depth compare is enabled in the sampler or not. If this bit is not present, linear filtering with depth compare disabled is unsupported and linear filtering with depth compare enabled is supported, but may compute the filtered value in an implementation-dependent manner which differs from the normal rules of linear filtering. The resulting value must be in the range [0,1] and should be proportional to, or a weighted average of, the number of comparison passes or failures." So VK_FORMAT_D32_SFLOAT_S8_UINT can be added to the list of filterable formats, since VK_FORMAT_D32_SFLOAT is already supported. Change-Id: Ida83c2630c920f3dc2c3e17853dfe91c835a4b7f Tests: dEQP-VK.*d32_sfloat_s8_uint* Bug: b/146563038 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/40028 Presubmit-Ready: Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Ben Clayton authored
This is an attempt to speed up build times and avoid the flaky timeouts we're seeing with the LLVM debug builds. Bug: b/147355576 Change-Id: I8dc83dd40a570849d02fe95822de534f0faae595 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39956Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Addresses the review comment: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39885/3/src/Pipeline/SpirvShader.cpp#422 Bug: b/145351270 Change-Id: I9a5ceefe048e63e0f52b7e9a15bd84f6bac16629 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39955Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-
Ben Clayton authored
Bug: b/147359661 Change-Id: I20e3b90b02327250d77a1006cea712022ad0f9d5 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39951Reviewed-by:
Alexis Hétu <sugoi@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
SwiftShader Regression Bot authored
Change-Id: I3ed2e5239fb255174fc523b59e7d038fae0ff984 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39954Reviewed-by:
Ben Clayton <bclayton@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
- 08 Jan, 2020 14 commits
-
-
Ben Clayton authored
Better to have a descriptive compiler error message than weird missing symbols. Bug: b/147359661 Change-Id: I66c0e5beb1e94f79210528867c990ab3e0f8b0ae Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39949Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
Bug: b/147359661 Change-Id: Ieb2de4d93ca617b190977e258023992f4a6d34c9 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39948Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
GCC 4.8 doesn't even understand --std=c++14 Bug: b/147359661 Change-Id: I5189d24638007ffe40e8903a382bccab9ef1c0fc Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39950Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Ben Clayton authored
Move `setActiveLaneMask()` from `SpirvShader::EmitState` to `SpirvShader`. Also capitalize the function name, as this is the more common style in this class. The debugger will want to expose which lanes are active, so we'll need to perform additional work whenever this is called. This is a no-op change. Bug: b/145351270 Change-Id: I294c1d7cda08faaa6b784db148b70a7c4d84d431 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39886 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Antonio Maiorano <amaiorano@google.com>
-
Ben Clayton authored
Previously `OpExtInst` and `OpExtInstImport` was hard-coded to only support `GLSL.std.450`. We will want to support `OpenCL.Debug.100`, so put in the plumbing to properly support other extensions. Bug: b/145351270 Change-Id: I60fbb8c45bb57b747067437641676946129b1251 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39885Reviewed-by:
Antonio Maiorano <amaiorano@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Strings are used for debug instructions. Bug: b/145351270 Change-Id: I6bc711c6c8b320bef6e5abbe9c4bda189fdafc0b Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39884Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Antonio Maiorano <amaiorano@google.com>
-
Ben Clayton authored
Instead of declaring a bunch of private forward declarations in SpirvShader, create a single Impl struct, which can hold all the forward declarations. The names of these forward declarations can then directly match the SpirvShader file name suffixes. The Impl struct can also be used later to store pointers to any internal state. This was proposed here: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38929/1/src/Pipeline/SpirvShaderGroup.cpp#23 Bug: b/145351270 Change-Id: Ie0de5e651fce9969fe0c24f51ae57652cade54d1 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39883Reviewed-by:
Antonio Maiorano <amaiorano@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Previously Frame was an Thread-internally managed structure, with special accessors for the variable containers of the top most frame. With this change, the Frame can be modified by calling `Thread::update()`. This allows for more powerful edits of the frame, while still preserving a mutex lock over the structure. Bug: b/145351270 Change-Id: I81e87884e26bdd6f22b47e8f71b37bbfc91c7830 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39882 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Antonio Maiorano <amaiorano@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Alexis Hetu authored
B10G11R11_UFLOAT blending is required in order for SwANGLE to expose GL_EXT_color_buffer_float. In this cl: - I added a minor readability improvement by storing "*Pointer<Int>(data + OFFSET(DrawData, colorPitchB[index]))" in a variable, "pitchB" since it's used in every case. - Added a constant for B10G11R11 masking - Added clamping in the floating point blending code for unsigned floating point formats - Fixed the VK_FORMAT_B10G11R11_UFLOAT_PACK32 output format which now takes the coverage mask into account - Included a minor followup cleanup in ShaderCore Bug: b/146223877 Tests: dEQP-VK.*b10g11r11* Change-Id: Ifb95f34a10cdbee9d185bc25feba0aeaca0d9e70 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39929Reviewed-by:
Chris Forbes <chrisforbes@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Presubmit-Ready: Alexis Hétu <sugoi@google.com> Tested-by:
Alexis Hétu <sugoi@google.com>
-
Ben Clayton authored
`vk::dbg::Reference` is a `vk::dbg::Value` that reads and writes from a value stored in memory. Used for displaying variables. Bug: b/145351270 Change-Id: I64146cb11b13267bef15d727af82a2256a9f25ce Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39881Reviewed-by:
Antonio Maiorano <amaiorano@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Presubmit-Ready: Ben Clayton <bclayton@google.com>
-
Chris Forbes authored
This is a step toward being able to claim support for storageImageExtendedFormats. Bug: b/146579770 Test: dEQP-VK.*r16g16* Change-Id: Iba658f39271f7e181c10c117e53c8e3dbdf37595 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39769 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Chris Forbes <chrisforbes@google.com>
-
Ben Clayton authored
SpirvShader doesn't actually do anything with this (yet). Extracts the dull plumbing from the meat of the SpirvShader debugger implementation. Bug: b/145351270 Change-Id: I1a648d8e463c428a232f711d0cc5dd0ecf94ad1e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38914Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
-
Ben Clayton authored
Add a debugger context and server to vk::Device, along with a getter for acquiring the debugger context. Generate a synthetic file containing all the Vulkan commands in the command buffer, and allow the debugger to single line step over these. All of this is no-op unless ENABLE_VK_DEBUGGER is defined at compile time, and the VK_DEBUGGER_PORT env var is set at run time. Bug: b/145351270 Change-Id: I8bea398a6c08d4cb23d76172dbca2a08ae109a6d Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/38913 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com> Tested-by:
Ben Clayton <bclayton@google.com>
-
Nicolas Capens authored
In file included from ../../third_party/swiftshader/third_party/llvm-subzero/lib/Support/Signals.cpp:183: ../../third_party/swiftshader/third_party/llvm-subzero/lib/Support/Windows/Signals.inc(220,13): error: unused function 'printStackTraceWithLLVMSymbolizer' [-Werror,-Wunused-function] static bool printStackTraceWithLLVMSymbolizer(llvm::raw_ostream &OS, ^ 1 error generated. Bug: chromium build fix Change-Id: I9ed9ac680fb5286fda0685788c6dbf76a4e34369 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39908Reviewed-by:Ben Clayton <bclayton@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:
Nicolas Capens <nicolascapens@google.com>
-
- 07 Jan, 2020 1 commit
-
-
Ben Clayton authored
The following tests are failing due to invalid SPIR-V merge blocks: dEQP-VK.spirv_assembly.instruction.graphics.loop.break_frag dEQP-VK.spirv_assembly.instruction.graphics.loop.break_vert dEQP-VK.spirv_assembly.instruction.graphics.loop.continue_frag dEQP-VK.spirv_assembly.instruction.graphics.loop.continue_vert dEQP-VK.spirv_assembly.instruction.graphics.loop.multi_block_loop_construct_frag dEQP-VK.spirv_assembly.instruction.graphics.loop.multi_block_loop_construct_vert dEQP-VK.spirv_assembly.instruction.graphics.loop.return_frag dEQP-VK.spirv_assembly.instruction.graphics.loop.return_vert dEQP-VK.spirv_assembly.instruction.graphics.opphi.out_of_order_frag dEQP-VK.spirv_assembly.instruction.graphics.opphi.out_of_order_vert dEQP-VK.spirv_assembly.instruction.graphics.switch_block_order.out_of_order_frag dEQP-VK.spirv_assembly.instruction.graphics.switch_block_order.out_of_order_vert This fix is currently working its way through the dEQP branches. Once it lands in 1.1.6, this change will be reverted and the dEQP SHA will be rolled forward. Change-Id: Ib69aa6aeb09c89519ff669281ef839a8555946bc Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/39878Tested-by:
Ben Clayton <bclayton@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
-