diff options
| author | Tim Foley <tfoleyNV@users.noreply.github.com> | 2018-02-02 08:49:04 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-02-02 08:49:04 -0800 |
| commit | 0360f81b9741ece65768a65731bd23455a3b7a96 (patch) | |
| tree | eab15cf6737cf091bf94d073e82c7b233f958e57 /tests/glsl/sascha-willems/texturearray | |
| parent | b034398ab12d3cc3a5fc04174688cb707404791d (diff) | |
Remove support for the -no-checking flag (#392)
* Remove support for the -no-checking flag
Fixes #381
Fixes #383
Work on #382
- No longer expose flag through API (`SLANG_COMPILE_FLAG_NO_CHECKING`) and command-line (`-no-checking`) options
- Remove all logic in `check.cpp` that was withholding diagnostics (including errors) when the no-checking mode was enabled
- Remove `HiddenImplicitCastExpr`, which was only created to support no-checking mode (it represented an implicit cast that our checking through was needed, but couldn't emit because it might be wrong)
- Remove logic for storing function bodies as raw token lists when checking is turned off. I'm leaving in the `UnparsedStmt` AST node in case we ever need/want to lazily parse and check function bodies down the line.
- Remove a few of the code-generation paths we had to contend with, but keep the comment about them in place.
- Remove GLSL-based tests that can't meaningfully work with the new approach.
- Fix other tests that used a GLSL baseline so that their GLSL compiles with `-pass-through glslang` instead of invoking `slang` with the `-no-checking` flag.
- Remove tests that were explicitly added to test the "rewriter + IR" path, since that is no longer supported.
There is more cleanup that can be done here, now that we know that AST-based rewrite and IR will never co-exist, but it is probably easier to deal with that as part of removing the AST-based rewrite path.
We've lost some test coverage here, but actually not too much if we consider that we are dropping GLSL input anyway.
* Fixup: test runner was mis-counting ignored tests
* Fixup: turn on dumping on test failure under Travis
* Fixup: enable extensions in Linux build of glslang
Diffstat (limited to 'tests/glsl/sascha-willems/texturearray')
| -rw-r--r-- | tests/glsl/sascha-willems/texturearray/instancing.frag | 16 | ||||
| -rw-r--r-- | tests/glsl/sascha-willems/texturearray/instancing.vert | 30 |
2 files changed, 0 insertions, 46 deletions
diff --git a/tests/glsl/sascha-willems/texturearray/instancing.frag b/tests/glsl/sascha-willems/texturearray/instancing.frag deleted file mode 100644 index b0e293e34..000000000 --- a/tests/glsl/sascha-willems/texturearray/instancing.frag +++ /dev/null @@ -1,16 +0,0 @@ -//TEST:COMPARE_GLSL: -#version 450 - -#extension GL_ARB_separate_shader_objects : enable -#extension GL_ARB_shading_language_420pack : enable - -layout (binding = 1) uniform sampler2DArray samplerArray; - -layout (location = 0) in vec3 inUV; - -layout (location = 0) out vec4 outFragColor; - -void main() -{ - outFragColor = texture(samplerArray, inUV); -}
\ No newline at end of file diff --git a/tests/glsl/sascha-willems/texturearray/instancing.vert b/tests/glsl/sascha-willems/texturearray/instancing.vert deleted file mode 100644 index ad738a59b..000000000 --- a/tests/glsl/sascha-willems/texturearray/instancing.vert +++ /dev/null @@ -1,30 +0,0 @@ -//TEST:COMPARE_GLSL: -#version 450 - -#extension GL_ARB_separate_shader_objects : enable -#extension GL_ARB_shading_language_420pack : enable - -layout (location = 0) in vec4 inPos; -layout (location = 1) in vec2 inUV; - -struct Instance -{ - mat4 model; - vec4 arrayIndex; -}; - -layout (binding = 0) uniform UBO -{ - mat4 projection; - mat4 view; - Instance instance[8]; -} ubo; - -layout (location = 0) out vec3 outUV; - -void main() -{ - outUV = vec3(inUV, ubo.instance[gl_InstanceIndex].arrayIndex.x); - mat4 modelView = ubo.view * ubo.instance[gl_InstanceIndex].model; - gl_Position = ubo.projection * modelView * inPos; -} |
