JBR-5973 Vulkan: Fix validation errors

JetBrains / JetBrainsRuntime

Runtime environment based on OpenJDK for running IntelliJ Platform-based products on Windows, macOS, and Linux

GNU General Public License v2.0

1.35k stars 198 forks source link

Closed YaaZ closed 2 months ago

YaaZ commented 2 months ago

Added proper synchronization and image layout transitions.
Refactored VKRenderer to hold per-device rendering context. Isolated surface rendering contexts.
Implemented reusing of command buffers and semaphores
Fixed surface resize, made surface initialization more robust.
Added on-demand pipeline creation for actual surface formats.
Added missing destruction logic.
Added macros for easy checking of return codes, logging with source code location.

YaaZ commented 2 months ago

Please disregard the first commit, there is a separate review for it: https://github.com/JetBrains/JetBrainsRuntime/pull/451

bourgesl commented 2 months ago

No problem, this PR is obsolete as you are making ndw PRs for smaller steps...

YaaZ commented 2 months ago

Nope, this one is not obsolete, it's a new one with smaller subset of changes.