Cleanup RenderPass logic for Pipeline

[spencer-lunarg]

closes https://github.com/KhronosGroup/Vulkan-ValidationLayers/issues/8202

The main goal was to try and make it clear in the pipeline validation where/when we have a renderpass/dynamic rendering/GPL being used

The core issue was we were mixing up

• rp_state == NULL when using something like GPL Vertex Input and there is no state OR there is a bad handle to VkRenderPass
• pipeline.GraphicsCreateInfo().renderPass == VK_NULL_HANDLE when the passed in value is null (which can be from Dynamic Rendering OR GPL)
• rp_state && rp_state.UsesDynamicRendering() which is when the pipeline passes in null, but we created a vvl::RenderPass at vkCmdBeginRendering and so there is a state object

Core issue is that vvl::RenderPass is used for dynamic rendering where there might not be an actual VkRenderPass VkHandle() under it

[ci-tester-lunarg]

CI Vulkan-ValidationLayers build queued with queue ID 208236.

[ci-tester-lunarg]

CI Vulkan-ValidationLayers build # 16957 running.

[ci-tester-lunarg]

CI Vulkan-ValidationLayers build # 16957 passed.

[ci-tester-lunarg]

CI Vulkan-ValidationLayers build queued with queue ID 208373.

[ci-tester-lunarg]

CI Vulkan-ValidationLayers build # 16958 running.

[ci-tester-lunarg]

CI Vulkan-ValidationLayers build # 16958 passed.