================
@@ -0,0 +1,30 @@
+// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.0-compute -x hlsl -o - %s
-verify
+
+// previously, this test would result in an error shown below on the line that
+// declares variable a in struct Eg9:
+// error: use of undeclared identifier
+// 'SV_DispatchThreadID'
+// This is because the annotation is parsed as if it was a c++ bit field, and
an identifier
+// that represents an integer is expected, but not found.
+
+// This test ensures that hlsl annotations are attempted to be parsed when
parsing struct decls.
+// Ideally, we'd validate this behavior by ensuring the annotation is parsed
and properly
+// attached as an attribute to the member in the struct in the AST. However,
in this case
+// this can't happen presently because there are other issues with annotations
on field decls.
+// This test just ensures we make progress by moving the validation error from
the realm of
+// C++ and expecting bitfields, to HLSL and a specialized error for the
recognized annotation.
+
+struct Eg9{
+// expected-error@+1{{attribute 'SV_DispatchThreadID' only applies to
parameter}}
+ int a : SV_DispatchThreadID;
----------------
llvm-beanz wrote:
No, my point was that the error you're checking for here isn't an error we can
rely on being persistent because that attribute _is_ allowed to apply to struct
members.
https://github.com/llvm/llvm-project/pull/96346
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
