safe_VkIndirectExecutionSetCreateInfoEXT is not safe

[ziga-lunarg]

safe_VkIndirectExecutionSetCreateInfoEXT contains VkIndirectExecutionSetInfoEXT, which is a union of pointers that are just copied and can be invalid later.

For example running CTS test dEQP-VK.dgc.ext.compute.layout.offset_execution_set_dispatch_shader_objects with VVL will crash

[spencer-lunarg]

Ok, so some background

#196

I proved for things like Descriptor Buffer the pointers (in the union) are required to be alive when used

For VkIndirectExecutionSetInfoEXT this is not the case, so we need to do the ugly union copying of things

[spencer-lunarg]

keeping this open, but KhronosGroup/Vulkan-ValidationLayers#12058 was the fix for VVL... basically we don't need the safe_VkIndirectExecutionSetCreateInfoEXT to begin with