From 7758625d3fea67e55e98e7e4103d56c9918365be Mon Sep 17 00:00:00 2001 From: ArielG-NV <159081215+ArielG-NV@users.noreply.github.com> Date: Fri, 29 Aug 2025 15:52:34 -0700 Subject: [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> --- source/slang/slang-parser.cpp | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) (limited to 'source/slang/slang-parser.cpp') diff --git a/source/slang/slang-parser.cpp b/source/slang/slang-parser.cpp index 1302975df..e71b6162c 100644 --- a/source/slang/slang-parser.cpp +++ b/source/slang/slang-parser.cpp @@ -7026,7 +7026,6 @@ static NodeBase* parseSizeOfExpr(Parser* parser, void* /*userData*/) static NodeBase* parseAlignOfExpr(Parser* parser, void* /*userData*/) { - // We could have a type or a variable or an expression AlignOfExpr* alignOfExpr = parser->astBuilder->create(); parser->ReadMatchingToken(TokenType::LParent); @@ -7058,6 +7057,17 @@ static NodeBase* parseCountOfExpr(Parser* parser, void* /*userData*/) return countOfExpr; } +static NodeBase* parseAddressOfExpr(Parser* parser, void* /*userData*/) +{ + // We could have a type or a variable or an expression + AddressOfExpr* addressOfExpr = parser->astBuilder->create(); + + parser->ReadMatchingToken(TokenType::LParent); + addressOfExpr->arg = parser->ParseExpression(); + parser->ReadMatchingToken(TokenType::RParent); + return addressOfExpr; +} + static NodeBase* parseTryExpr(Parser* parser, void* /*userData*/) { auto tryExpr = parser->astBuilder->create(); @@ -9648,6 +9658,7 @@ static const SyntaxParseInfo g_parseSyntaxEntries[] = { _makeParseExpr("sizeof", parseSizeOfExpr), _makeParseExpr("alignof", parseAlignOfExpr), _makeParseExpr("countof", parseCountOfExpr), + _makeParseExpr("__getAddress", parseAddressOfExpr), }; ConstArrayView getSyntaxParseInfos() -- cgit v1.2.3