================
@@ -0,0 +1,13 @@
+// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.3-library -x hlsl 
-fsyntax-only %s -verify
+
+// expected-no-diagnostics
+
+// Test that we have consistent behaviour for comma parsing. Namely:
+// - a single trailing comma is allowed after any parameter
+// - a trailing comma is not required
+
+[RootSignature("CBV(b0, flags = DATA_VOLATILE,), 
DescriptorTable(Sampler(s0,),),")]
----------------
inbelic wrote:

The intended behaviour is to reject multiple commas. It was just quite hard to 
decouple from the implementation of a new diag done 
[here](https://github.com/llvm/llvm-project/pull/147800). And we will get the 
multiple error warning for free because of this change there:

https://github.com/llvm/llvm-project/blob/70448a3e3daae9ebac71df5b995bc93025b7a644/clang/lib/Parse/ParseHLSLRootSignature.cpp#L62-L67

and will be tested here:

https://github.com/llvm/llvm-project/blob/7ec7e32d2ac4945a489d5463b9fb700b0cceff9d/clang/test/SemaHLSL/RootSignature-err.hlsl#L117

For context, the original draft of this pr, had both an improvement of 
diagnostic but I wanted to trim it down as much as possible to just be a fix, 
so that we could get it in before the other pr.

https://github.com/llvm/llvm-project/pull/147350
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to