https://github.com/V-FEXrt closed
https://github.com/llvm/llvm-project/pull/141086
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
V-FEXrt wrote:
Tracking bug! https://github.com/llvm/llvm-project/issues/141842
https://github.com/llvm/llvm-project/pull/141086
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bogner approved this pull request.
Looks good (noting that we're matching the pre-existing buggy behaviour of
StructuredBuffer and will follow up to fix that for both)
https://github.com/llvm/llvm-project/pull/141086
___
cfe-commits
https://github.com/inbelic approved this pull request.
LGTM given we have a way of tracking making sure it will be read-only
https://github.com/llvm/llvm-project/pull/141086
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org
V-FEXrt wrote:
@inbelic From Justin, looks like it doesn't :) (and this is a bug)
https://hlsl.godbolt.org/z/sqMMffh6K
I'm going to spin that off into its own task separate from this one so this
doesn't get bogged down
https://github.com/llvm/llvm-project/pull/141086
_
https://github.com/inbelic commented:
How does this/will this handle enforcing that it is a read-only buffer?
https://github.com/llvm/llvm-project/pull/141086
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
https://github.com/V-FEXrt updated
https://github.com/llvm/llvm-project/pull/141086
>From 9868eef5e0752545c88e44ba2fb5d1929feb1e1a Mon Sep 17 00:00:00 2001
From: Ashley Coleman
Date: Thu, 22 May 2025 09:19:38 -0600
Subject: [PATCH 1/2] [HLSL] Add Buffer def to frontend
---
clang/lib/Sema/HLSL
https://github.com/bogner commented:
We should also add AST tests in clang/test/AST/HLSL/TypedBuffers-AST.hlsl
https://github.com/llvm/llvm-project/pull/141086
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailm