ARB_gl_spirv: include SPIR-V names if available
SPIR-V variable names are optional, so everything need to work without it. ARB_gl_spirv extension allows to ignore all names. In order to simplify the implementation, and to ensure that everything really worked without names, we were explicitly ignoring the names (so internally not saving it, and setting any name to NULL).
But that means that in case of any error, names that could be used to debug the problem are missing.
Now that ARB_gl_spirv support has landed, so everything works reasonably well without names, it would be good to recheck if makes sense to bring them back.
Note that this could be messy. As names are optional, nothing prevents to use different names for the same resource on different stages (for example, for a ssbo with binding=3, using different names on the vertex and fragment shader). Also, nothing prevents to have one stage with names, and other without it.
Also, right now glsl types got included on hash tables, and the hash id is created with the name, as on GLSL it can be used to differentiate a type from other. But that is not the case here.
So, there is the possibility that names coming from SPIR-V needs to be considered different to names coming from GLSL. And perhaps a "debug-name" needs to be added to mesa glsl type, as in the end, their behaviour is different to GLSL "real name".