Closed billhollings closed 1 year ago
It seems to me that the workaround here is straightforward - just always output "position" from VS. The justification to introduce a flag for it is therefore weak - it would incur work for the CTS, for spec complexity, etc, while a fix in SPIRV-Cross is simple.
This has been fixed in SPIRV-Cross/MoltenVK.
Um, I still lack permission to close these bugs...
Um, I still lack permission to close these bugs...
Sorry about that oversight. I've added you as an admin collaborator to this repo now. Try again when you get a chance.
Metal requires a position output from all vertex shaders. Some CTS tests fail when the shader does not include a position output, but only in certain structural situations.
This is likely fixable with effort within the implementations.
In the meantime, these complex use cases can be disabled in CTS and reported as Not Supported, by adding a
VkPhysicalDevicePortabilitySubsetFeaturesKHR:: shaderNoPositionC0E1
capability.This affects 8 CTS tests: