skia2/tests/sksl/intrinsics/MixBool.asm.frag

### Compilation failed:

error: SPIR-V validation error: Invalid instruction OpExtInst starting at word 577: expected no more operands after 6 words, but stated word count is 8.
SPIR-V validation error: Invalid instruction OpExtInst starting at word 577: expected no more operands after 6 words, but stated word count is 8.

1 error
Implement additional Runtime Effect intrinsic tests. Tested in dm: - max(halfN, halfN) - max(halfN, half) - min(halfN, halfN) - min(halfN, half) - clamp(halfN, halfN, halfN) - clamp(halfN, half, half) - mix(halfN, halfN, halfN) - mix(halfN, halfN, half) Compiled outputs only (not in ES2): - max(intN, intN) - max(intN, int) - min(intN, intN) - min(intN, int) - clamp(intN, intN, intN) - clamp(intN, int, int) - mix(halfN, halfN, boolN) - broken in Metal/SPIR-V (skia:11222) - mix(intN, intN, boolN) - broken in Metal/SPIR-V (skia:11222) Change-Id: Iaafc5429b16d2a8710b9d171ae281c268c0fd70d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/361438 Commit-Queue: John Stiles <johnstiles@google.com> Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Auto-Submit: John Stiles <johnstiles@google.com> 2021-01-29 16:51:47 +00:00			`### Compilation failed:`

Revert "Refactored SPIR-V RelaxedPrecision handling" This reverts commit 9e476b798f03930b6bc7fb6eacab574f197e172d. Reason for revert: Angry Vulkan bots Original change's description: > Refactored SPIR-V RelaxedPrecision handling > > The RelaxedPrecision decoration is now handled by nextId(), to make it > easier to see all spots where a RelaxedPrecision decoration might be > necessary. The goal of this initial refactor is not to actually fix the > issues with RelaxedPrecision decorations, but rather to lay the > groundwork for doing so in followup CLs. > > The initial intent of this change was to not affect the SPIR-V at all, > saving modifications for followups, but there ended up being three kinds > of changes to the output: > > 1. Doing things at nextId() time rather than later means some > decorations move to an earlier spot in the output. This results in > diffs, but should not cause any behavioral changes. > 2. We were incorrectly tagging bools as RelaxedPrecision in some > situations. By funneling things through fewer code paths, the refactor > would have caused this to happen in even more situations, and the code > responsible for the bug was being rewritten in this CL anyway, so it > seemed worth just fixing the issue as part of this change. > 3. Funneling things through fewer code paths ended up adding > (correct) RelaxedPrecision modifiers to binary operations that had > previously been missing them. It seemed better to just let it happen > than to try to maintain bug-for-bug compatibility with the previous > approach. > > Change-Id: Ia9654d6b5754e2c797e02226660cb618c9189b36 > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/384318 > Commit-Queue: Ethan Nicholas <ethannicholas@google.com> > Reviewed-by: Brian Osman <brianosman@google.com> TBR=brianosman@google.com,ethannicholas@google.com,johnstiles@google.com Change-Id: I9ada728e5fd5798bc1179640560c2e6045b7efd1 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://skia-review.googlesource.com/c/skia/+/385158 Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: Ethan Nicholas <ethannicholas@google.com> 2021-03-15 21:24:32 +00:00			`error: SPIR-V validation error: Invalid instruction OpExtInst starting at word 577: expected no more operands after 6 words, but stated word count is 8.`
			`SPIR-V validation error: Invalid instruction OpExtInst starting at word 577: expected no more operands after 6 words, but stated word count is 8.`
Implement additional Runtime Effect intrinsic tests. Tested in dm: - max(halfN, halfN) - max(halfN, half) - min(halfN, halfN) - min(halfN, half) - clamp(halfN, halfN, halfN) - clamp(halfN, half, half) - mix(halfN, halfN, halfN) - mix(halfN, halfN, half) Compiled outputs only (not in ES2): - max(intN, intN) - max(intN, int) - min(intN, intN) - min(intN, int) - clamp(intN, intN, intN) - clamp(intN, int, int) - mix(halfN, halfN, boolN) - broken in Metal/SPIR-V (skia:11222) - mix(intN, intN, boolN) - broken in Metal/SPIR-V (skia:11222) Change-Id: Iaafc5429b16d2a8710b9d171ae281c268c0fd70d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/361438 Commit-Queue: John Stiles <johnstiles@google.com> Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Auto-Submit: John Stiles <johnstiles@google.com> 2021-01-29 16:51:47 +00:00
			`1 error`