dan sinclair

Potential parsing conflict with const literals and postfix expressions.

6

Currently the grammar contains `const_literal` as part of the `primary_expression` rule. This is called by `singular_expression`. This means it's valid in the grammar to parse `1.foo` as the `const_literal` (1)...

Add precedence rules description into spec

3

In https://github.com/gpuweb/gpuweb/issues/1146 we worked out the precedence rules and the grammar was updated. There is no text (or diagram) explaining the rules outside of reading the grammar. We should add...

For an array with an override-able size the comment in the example could be clarified. Currently we say: ``` // An invalid example, because the overridable element count may only...

New job posting notifications?

10

The mockup for the new career page has a get notified section. Do we want to add this functionality to the site? How should it work? Is the privacy policy...

Test failure with tip-of-tree spirv-tools

4

When building glslang with tip-of-tree spirv-tools (which contains https://github.com/KhronosGroup/SPIRV-Tools/commit/784b064f90a597d2478064161438f4986485b5d4) the glslang tests fail with: `out/Debug/gtests/glslangtests --gtest_filter="Glsl/CompileVulkanToSpirvTest.FromFile/spv_bufferhandle3_frag"` ``` The following warnings/errors occurred: error: SPIRV-Tools Validation Errors error: [VUID-StandaloneSpirv-Input-09557] Input/Output interface variable...

Use enum tokens when they are the only possible grammar usage.

17

When moving various parts of the grammar over to be general we moved all of the enums and builtins to be shadowable. We then do lookups to determine which variable...

In the [Functions](https://www.w3.org/TR/WGSL/#functions) section of the spec we say: `A built-in function is provided by the WGSL implementation, and is always available to a WGSL module.` While this was true...

wgsl: validate struct parsing

1

* [x] `struct a {}` -- invalid (empty struct) * [ ] `struct { a: i32, }` -- invalid no name * [ ] invalid duplicate struct name ``` struct...

wgsl: Validate variable and value declaration parsing

1

* [ ] `var a;` * [x] `var a:i32;` -- passes * [ ] `var i32:f32;` * [ ] `var :i32;` * [ ] `var a:invalid;` * [ ] `var...

wgsl: validate parsing of type aliases

1

* [ ] type alias to an existing type alias passes * [ ] Existing alias before new alias * [ ] Existing alias after new alias * [ ]...