Builder decorate methods don't have a way to specify extra decorator operands about rspirv

[antiagainst]

Ouch! I should definitely add tests for the builder. The decoder and parser are extensively covered by tests, but the builder I delayed it a little bit, thinking that they are basically automatically generated and should be fine. But apparently not.

Yep, the grammar doesn't directly encode additional parameters to decorations in the OpDecorate instruction (or its siblings). They are encoded in the Decoration operand kind.

Thanks a lot for reporting! Will fix soon. :)

[antiagainst]

What is interesting is that if you check 8411ea6, the code removed were manually written and they did have the additional parameters. :)

[tedsta]

Ahhh haha. I do like your code generation though. Pretty slick.

[antiagainst]

Will push a new release soon

[tedsta]

Sweet, thank you!

[antiagainst]

Done. https://crates.io/crates/rspirv/0.3.3 :)