KhronosGroup / SPIRV-Tools

Apache License 2.0
1.08k stars 555 forks source link

Maybe: Check type of OpSampledImage image component vs result type's image component #838

Open dneto0 opened 7 years ago

dneto0 commented 7 years ago

See test case in https://github.com/KhronosGroup/glslang/issues/1073

johnkslang commented 7 years ago

Since it is not yet entirely clear the spec should not be changed here, can this one be prioritized behind missing validator pieces that are unquestionably needed but have been missing for some time now?

dneto0 commented 7 years ago

Agreed. Deprioritized to the lowest level, and added a "Maybe" in the title.