diff options
| author | Tim Foley <tfoleyNV@users.noreply.github.com> | 2018-02-03 07:30:54 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-02-03 07:30:54 -0800 |
| commit | 662f43fff6721c6cd013a8f1b2639c2e29fe6be3 (patch) | |
| tree | 9e57c4a9f1a922418fbae2390ee1998984a6ea26 /tests/bugs | |
| parent | 58475a8aa42284722a3763aa3bde49f2fa40366e (diff) | |
Remove non-IR codegen paths (#398)
The basic change is simple: remove support for all code generation paths other than the IR.
There is a lot of vestigial code left, but the main logic in `ast-legalize.*` is gone.
Doing this breaks a *lot* of tests, for various reasons:
- We can no longer guarantee exactly matching DXBC or SPIR-V output after things pass through out IR
- Many builtins don't have matching versions defined for GLSL output via IR (even when they had versions defined via the earlier approach that worked with the AST)
- A lot of code creates intermediate values of opaque types in the IR, which turn into opaque-type temporaries that aren't allowed (this breaks many GLSL tests, but also some HLSL)
I implemented some small fixes for issues that I could get working in the time I had, but most of the above are larger than made sense to fix in this commit.
For now I'm disabling the tests that cause problems, but we will need to make a concerted effort to get things working on this new substrate if we are going to make good on our goals.
Diffstat (limited to 'tests/bugs')
| -rw-r--r-- | tests/bugs/do-loop.hlsl | 2 | ||||
| -rw-r--r-- | tests/bugs/gh-103.slang | 2 | ||||
| -rw-r--r-- | tests/bugs/gh-122.slang | 4 | ||||
| -rw-r--r-- | tests/bugs/gh-133.slang | 2 | ||||
| -rw-r--r-- | tests/bugs/gh-333.slang | 2 | ||||
| -rw-r--r-- | tests/bugs/import-with-error.slang | 2 | ||||
| -rw-r--r-- | tests/bugs/nested-generics-call.slang | 2 | ||||
| -rw-r--r-- | tests/bugs/nested-generics-method-call.slang | 2 | ||||
| -rw-r--r-- | tests/bugs/split-nested-types.hlsl | 3 |
9 files changed, 11 insertions, 10 deletions
diff --git a/tests/bugs/do-loop.hlsl b/tests/bugs/do-loop.hlsl index eea62e92a..de98a9765 100644 --- a/tests/bugs/do-loop.hlsl +++ b/tests/bugs/do-loop.hlsl @@ -1,4 +1,4 @@ -//TEST:COMPARE_HLSL: -profile vs_5_0 -target dxbc-assembly +//TEST_DISABLED:COMPARE_HLSL: -profile vs_5_0 -target dxbc-assembly // Check output for `do` loops diff --git a/tests/bugs/gh-103.slang b/tests/bugs/gh-103.slang index 4e9765fb3..b89f38098 100644 --- a/tests/bugs/gh-103.slang +++ b/tests/bugs/gh-103.slang @@ -1,4 +1,4 @@ -//TEST:COMPARE_HLSL:-no-mangle -use-ir -profile ps_4_0 -entry main +//TEST:COMPARE_HLSL:-no-mangle -profile ps_4_0 -entry main // Ensure that matrix-times-scalar works diff --git a/tests/bugs/gh-122.slang b/tests/bugs/gh-122.slang index 1a011df37..c40692e6c 100644 --- a/tests/bugs/gh-122.slang +++ b/tests/bugs/gh-122.slang @@ -1,4 +1,6 @@ -//TEST:CROSS_COMPILE: -profile ps_5_0 -entry main -target spirv-assembly +//TEST_DISABLED:CROSS_COMPILE: -profile ps_5_0 -entry main -target spirv-assembly + +// Note: disabled, pending adding an IR-based definition for `GetDimensions`. // Ensure that `GetDimensions` with `mipCount` output works // on a `Texture2D` diff --git a/tests/bugs/gh-133.slang b/tests/bugs/gh-133.slang index 56a896ec9..f0e9d0d4e 100644 --- a/tests/bugs/gh-133.slang +++ b/tests/bugs/gh-133.slang @@ -1,4 +1,4 @@ -//TEST:CROSS_COMPILE: -profile ps_5_0 -entry main -target spirv-assembly +//TEST_DISABLED:CROSS_COMPILE: -profile ps_5_0 -entry main -target spirv-assembly // Ensure that an integer output from // a fragment shader doesn't get a `flat` qualifier diff --git a/tests/bugs/gh-333.slang b/tests/bugs/gh-333.slang index 8028c9cd9..fdc478950 100644 --- a/tests/bugs/gh-333.slang +++ b/tests/bugs/gh-333.slang @@ -1,4 +1,4 @@ -//TEST:COMPARE_HLSL:-profile ps_5_0 -entry main +//TEST:COMPARE_HLSL:-no-mangle -profile ps_5_0 -entry main // Ensure declaration order in output is correct diff --git a/tests/bugs/import-with-error.slang b/tests/bugs/import-with-error.slang index f50bb1adb..e54f9e727 100644 --- a/tests/bugs/import-with-error.slang +++ b/tests/bugs/import-with-error.slang @@ -1,4 +1,4 @@ -//TEST:SIMPLE:-use-ir +//TEST:SIMPLE: // Confirm that we correctly issue a diagnostic when // we `import` a module that has some errors in it. diff --git a/tests/bugs/nested-generics-call.slang b/tests/bugs/nested-generics-call.slang index 2c6df59c6..438cbb86c 100644 --- a/tests/bugs/nested-generics-call.slang +++ b/tests/bugs/nested-generics-call.slang @@ -1,4 +1,4 @@ -//TEST(compute):COMPARE_COMPUTE:-xslang -use-ir +//TEST(compute):COMPARE_COMPUTE: //TEST_INPUT:ubuffer(data=[0 0 0 0], stride=4):dxbinding(0),glbinding(0),out RWStructuredBuffer<float> outputBuffer; diff --git a/tests/bugs/nested-generics-method-call.slang b/tests/bugs/nested-generics-method-call.slang index d1e80da57..a09a25855 100644 --- a/tests/bugs/nested-generics-method-call.slang +++ b/tests/bugs/nested-generics-method-call.slang @@ -1,4 +1,4 @@ -//TEST(compute):COMPARE_COMPUTE:-xslang -use-ir +//TEST(compute):COMPARE_COMPUTE: //TEST_INPUT:ubuffer(data=[0 0 0 0], stride=4):dxbinding(0),glbinding(0),out RWStructuredBuffer<float> outputBuffer; diff --git a/tests/bugs/split-nested-types.hlsl b/tests/bugs/split-nested-types.hlsl index 210c119df..0a8a8f9ff 100644 --- a/tests/bugs/split-nested-types.hlsl +++ b/tests/bugs/split-nested-types.hlsl @@ -1,5 +1,4 @@ -// array-size-static-const.hlsl -//TEST:COMPARE_HLSL: -profile ps_5_0 -target dxbc-assembly +//TEST:COMPARE_HLSL:-no-mangle -profile ps_5_0 -target dxbc-assembly #ifdef __SLANG__ import split_nested_types; |
