Fix for a generic definition, followed by a declaration with target intrisic causing a crash (#1329)

* * Make a 'definition' if a function has a body or a target intrinsic defined
* Added test for this situation.

* Fix tab.

* Fix single-target-intrisic.slang expected output.

Co-authored-by: Tim Foley <tfoleyNV@users.noreply.github.com>

2 files changed, 23 insertions, 0 deletions

diff --git a/tests/diagnostics/single-target-intrinsic.slang b/tests/diagnostics/single-target-intrinsic.slang
new file mode 100644
index 000000000..e8ff0ed1a
--- /dev/null
+++ b/tests/diagnostics/single-target-intrinsic.slang
@@ -0,0 +1,16 @@
+//DIAGNOSTIC_TEST(windows):SIMPLE:
+
+
+T doThing<T>(T in)
+{
+    return in;
+}
+
+__target_intrinsic(hlsl, "doSomethingIntrinsically($0)")
+T doThing<T>(T in);
+
+void test()
+{
+    int a = 5;
+    int b = doThing(a);
+}
\ No newline at end of file
diff --git a/tests/diagnostics/single-target-intrinsic.slang.expected b/tests/diagnostics/single-target-intrinsic.slang.expected
new file mode 100644
index 000000000..82068f17d
--- /dev/null
+++ b/tests/diagnostics/single-target-intrinsic.slang.expected
@@ -0,0 +1,7 @@
+result code = -1
+standard error = {
+tests/diagnostics/single-target-intrinsic.slang(10): error 30201: function 'doThing' already has a body
+tests/diagnostics/single-target-intrinsic.slang(4): note: see previous definition of 'doThing'
+}
+standard output = {
+}