From 4fb3b10b81cf8c976ebd1ebb7fcde7708f022957 Mon Sep 17 00:00:00 2001 From: Yong He Date: Wed, 29 Nov 2023 11:29:14 -0800 Subject: Improve generic type argument inference. (#3370) * Improve generic type argument inference. * Fix. * Fix. --------- Co-authored-by: Yong He --- tests/diagnostics/bad-operator-call.slang | 4 +-- .../generics/generic-overload-disambiguation.slang | 42 ++++++++++++++++++++++ .../language-feature/generics/vector-generic.slang | 19 ++++++++++ 3 files changed, 63 insertions(+), 2 deletions(-) create mode 100644 tests/language-feature/generics/generic-overload-disambiguation.slang create mode 100644 tests/language-feature/generics/vector-generic.slang (limited to 'tests') diff --git a/tests/diagnostics/bad-operator-call.slang b/tests/diagnostics/bad-operator-call.slang index 2764d27ae..2e0a196a2 100644 --- a/tests/diagnostics/bad-operator-call.slang +++ b/tests/diagnostics/bad-operator-call.slang @@ -14,7 +14,7 @@ void test() { int a; S b; - // CHECK:{{.*}}.slang(18): error {{.*}}: no overload for '+=' applicable to arguments of type (int, S) + // CHECK:{{.*}}.slang(18): error {{.*}}: a += b; // CHECK:{{.*}}.slang(20): error {{.*}}: no overload for '+' applicable to arguments of type (int, S) a = a + b; @@ -31,4 +31,4 @@ void test() d += c; // CHECK:{{.*}}.slang(33): error {{.*}}: no overload for '+' applicable to arguments of type (vector, vector) d = c + d; -} \ No newline at end of file +} diff --git a/tests/language-feature/generics/generic-overload-disambiguation.slang b/tests/language-feature/generics/generic-overload-disambiguation.slang new file mode 100644 index 000000000..57d7a6d7e --- /dev/null +++ b/tests/language-feature/generics/generic-overload-disambiguation.slang @@ -0,0 +1,42 @@ +//TEST(compute):COMPARE_COMPUTE(filecheck-buffer=CHECK): -shaderobj -output-using-type +//TEST(compute):COMPARE_COMPUTE(filecheck-buffer=CHECK): -vk -shaderobj -output-using-type + +// Test that calls to overloaded generic functions can be resolved to prefer the +// generic candidate with more specialized constraints. +interface IBase +{ + float get(); +} +interface IDerived : IBase +{ + +} +float process(T v) +{ + return 0.0; +} +float process(T v) +{ + return v.get(); +} + +float process(T v) +{ + return v.get() + 1.0; +} + +struct D : IDerived +{ + float get() { return 1.0; } +} + +//TEST_INPUT:ubuffer(data=[0 0 0 0], stride=4):out,name=outputBuffer +RWStructuredBuffer outputBuffer; + +[numthreads(1,1,1)] +void computeMain() +{ + D d; + outputBuffer[0] = process(d); + // CHECK: 2.0 +} diff --git a/tests/language-feature/generics/vector-generic.slang b/tests/language-feature/generics/vector-generic.slang new file mode 100644 index 000000000..6bcd88018 --- /dev/null +++ b/tests/language-feature/generics/vector-generic.slang @@ -0,0 +1,19 @@ +//TEST(compute):COMPARE_COMPUTE(filecheck-buffer=CHECK): -shaderobj -output-using-type +//TEST(compute):COMPARE_COMPUTE(filecheck-buffer=CHECK): -vk -shaderobj -output-using-type + +// Test that generic argument inference works when passing a scalar to a generic vector parameter. +T process(vector v) +{ + return v[0]; +} + +//TEST_INPUT:ubuffer(data=[0 0 0 0], stride=4):out,name=outputBuffer +RWStructuredBuffer outputBuffer; + +[numthreads(1,1,1)] +void computeMain() +{ + float a = 1.0; + outputBuffer[0] = process(a); + // CHECK: 1.0 +} -- cgit v1.2.3