Keenuts wrote:

> We should have tests for some cases like:
> 
> ```
> struct S {
>   float4 f0 : SV_Position;
>   float4 f1;  // error due to missing semantic here!
> };
> 
> float4 main(S s) { }
> 
> // Semantics on return objects are required but not supported yet (right?)
> float4 main2(float4 p : SV_POSITION, float4 f) { } // error for missing 
> semantic on `f`
> ```

Yes, I'll add several sema checks now that those are validated in sema.

https://github.com/llvm/llvm-project/pull/159047
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to