summaryrefslogtreecommitdiffstats
path: root/tests/diagnostics
diff options
context:
space:
mode:
authorArielG-NV <159081215+ArielG-NV@users.noreply.github.com>2025-08-29 15:52:34 -0700
committerGitHub <noreply@github.com>2025-08-29 22:52:34 +0000
commit7758625d3fea67e55e98e7e4103d56c9918365be (patch)
tree2ed40aeb4d16262866e5540dad1a519951b5f772 /tests/diagnostics
parent450ef7934c1adfdf4a3a3c72967de3c5798a020d (diff)
[CBP] Pointer frontend changes + groupshared pointer support (#7848)
Resolves #7628 Resolves: #8197 Primary Goals: 1. Add `Access` to pointer 2. AddressSpace::GroupShared support for pointers (SPIR-V) 3. Add `__getAddress()` to replace `&` * `&` is not updated to `require(cpu)` since slangpy uses `&`. This means we must: (1) merge PR; (2) replace `&` with `__getAddress()`; (3) add `require(cpu)` to `&` Changes: * Added to `Ptr` the `Access` generic argument & logic (for `Access::Read`). * Moved the generic argument `AddressSpace` from `Ptr` to the end of the type. * Added pointer casting support between any `Ptr` as long as the `AddressSpace` is the same * Disallow globallycoherent T* and coherent T* * Disallow const T*, T const*, and const T* * Fixed .natvis display of `ConstantValue` `ValOperandNode` * Support generic resolution of type-casted integers * Added `VariablePointer` emitting for spirv + other minor logic needed for groupshared pointers Breaking Changes: * Anyone using the `AddressSpace` of `Ptr` will now have to account for the `Access` argument * we disallow various syntax paired with `Ptr` and `T*` --------- Co-authored-by: slangbot <186143334+slangbot@users.noreply.github.com>
Diffstat (limited to 'tests/diagnostics')
-rw-r--r--tests/diagnostics/invalid-constant-pointer-taking.slang16
1 files changed, 11 insertions, 5 deletions
diff --git a/tests/diagnostics/invalid-constant-pointer-taking.slang b/tests/diagnostics/invalid-constant-pointer-taking.slang
index 349f8cc25..658a84b1b 100644
--- a/tests/diagnostics/invalid-constant-pointer-taking.slang
+++ b/tests/diagnostics/invalid-constant-pointer-taking.slang
@@ -1,4 +1,4 @@
-//DIAGNOSTIC_TEST:SIMPLE(filecheck=CHECK): -stage compute -entry computeMain -target spirv
+//TEST:SIMPLE(filecheck=CHECK): -stage compute -entry computeMain -target spirv
RWStructuredBuffer<float> mutable_float_buffer;
RWStructuredBuffer<uint> mutable_uint_buffer;
@@ -6,18 +6,24 @@ RWStructuredBuffer<uint> mutable_uint_buffer;
StructuredBuffer<float> constant_float_buffer;
StructuredBuffer<uint> constant_uint_buffer;
+// We do not allow taking a pointer from a StructuredBuffer/RWStructuredBuffer.
[shader("compute")]
[numthreads(1,1,1)]
void computeMain(uint3 threadId : SV_DispatchThreadID)
{
- float* mutablePtr = &mutable_float_buffer[threadId.x];
+ float* mutablePtr1 = &mutable_float_buffer[threadId.x];
+
+ // CHECK: ([[# @LINE+1]]): error 31160
+ float* mutablePtr2 = __getAddress(mutable_float_buffer[threadId.x]);
InterlockedAdd(mutable_uint_buffer[threadId.x], 1);
// Constant pointers arent a thing in slang
- // CHECK: error 30078:
- float* ptr = &constant_float_buffer[threadId.x];
+ // CHECK: ([[# @LINE+1]]): error 30079:
+ float* ptr1 = &constant_float_buffer[threadId.x];
+ // CHECK: ([[# @LINE+1]]): error 31160
+ float* ptr2 = __getAddress(constant_float_buffer[threadId.x]);
InterlockedAdd(constant_uint_buffer[0], 1);
-} \ No newline at end of file
+}