-
Notifications
You must be signed in to change notification settings - Fork 403
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
gpu: Defer GPL instrumenation until linking #8468
gpu: Defer GPL instrumenation until linking #8468
Conversation
CI Vulkan-ValidationLayers build queued with queue ID 245030. |
CI Vulkan-ValidationLayers build # 17336 running. |
CI Vulkan-ValidationLayers build # 17336 failed. |
58fa9bc
to
d5dd9d1
Compare
CI Vulkan-ValidationLayers build queued with queue ID 246414. |
CI Vulkan-ValidationLayers build # 17348 running. |
CI Vulkan-ValidationLayers build # 17348 passed. |
d5dd9d1
to
aa434e2
Compare
CI Vulkan-ValidationLayers build queued with queue ID 258551. |
CI Vulkan-ValidationLayers build # 17500 running. |
CI Vulkan-ValidationLayers build # 17500 failed. |
aa434e2
to
71f2e73
Compare
CI Vulkan-ValidationLayers build queued with queue ID 258572. |
CI Vulkan-ValidationLayers build # 17501 running. |
CI Vulkan-ValidationLayers build # 17501 passed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, no need to address my complaints in the test now, it was more of a global rant about some problems I think we have
I need to make the other nit updates (also realized one more test I want to add) so happy to do it... but agree it is quite a mess and we only copy-and-paste it around like wild fire |
71f2e73
to
52ca451
Compare
CI Vulkan-ValidationLayers build queued with queue ID 258839. |
CI Vulkan-ValidationLayers build # 17504 running. |
CI Vulkan-ValidationLayers build # 17504 passed. |
For #8459
Currently a little rough, if this works well, will want to cleanup the somewhat duplicate code path (already wanted to do it with shader object path)