skia2/tests/sksl/runtime_errors/IllegalStatements.skvm

### Compilation failed:

error: 3: discard statement is only permitted in fragment shaders
error: 5: do-while loops are not supported
error: 7: while loops are not supported
error: 9: switch statements are not supported
error: 9: function 'switch_stmt' can exit without returning a value
5 errors
Move all runtime effect error tests to runtime_errors Also renamed Discard to IllegalStatements, and added testing of while and do loops. Change-Id: Ibacf69131267a0436808e2e022ad126704af16ef Reviewed-on: https://skia-review.googlesource.com/c/skia/+/353706 Commit-Queue: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: Brian Osman <brianosman@google.com> Reviewed-by: John Stiles <johnstiles@google.com> 2021-01-14 16:40:56 +00:00			`### Compilation failed:`

SkSL: Turn DSL parser off again Several fuzzer issues, and one Chromium issue that's blocking the roll. Bug: chromium:1246795 Bug: skia:12423 Change-Id: I00370b74569b447e543d9a1f22c588eb493063da Reviewed-on: https://skia-review.googlesource.com/c/skia/+/445960 Reviewed-by: John Stiles <johnstiles@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Auto-Submit: Brian Osman <brianosman@google.com> 2021-09-07 13:36:08 +00:00			`error: 3: discard statement is only permitted in fragment shaders`
Fix line numbers of several DSLParser errors Change-Id: I82e0d29fbcb5be93fe20a1fb442e2a429e6cab8e Bug: skia:12411 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/445644 Reviewed-by: John Stiles <johnstiles@google.com> Commit-Queue: John Stiles <johnstiles@google.com> 2021-09-03 21:00:09 +00:00			`error: 5: do-while loops are not supported`
			`error: 7: while loops are not supported`
			`error: 9: switch statements are not supported`
Generate more diagnostics from IRGenerator, when given bad code. Two minor changes: - Converting a Block with bad statements will now generate a partial block instead of nullptr. The change mirrors how DSL behaves; functions containing invalid statements will now be created and added to the program. Previously, we would discard a function definition with any invalid statements inside; this prevented duplicate-function-definition errors from appearing. - Converting a return with a bad expression will now generate a poisoned return instead of nullptr. This change improves diagnostics for functions with invalid return statements. If we eliminate the return statements (by returning null), we report bad return statements as "function can exit without returning a value" (which is confusing). Change-Id: I6d998d5c50585f8d96bb7e3cb7f59b63125d6a62 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/446325 Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Reviewed-by: Brian Osman <brianosman@google.com> 2021-09-07 19:22:02 +00:00			`error: 9: function 'switch_stmt' can exit without returning a value`
			`5 errors`