Opened 14 months ago
Closed 14 months ago
#18995 closed enhancement (fixed)
Vulkan-Headers Vulkan-Loader 1.3.273
Reported by: | Bruce Dubbs | Owned by: | Douglas R. Reno |
---|---|---|---|
Priority: | normal | Milestone: | 12.1 |
Component: | BOOK | Version: | git |
Severity: | normal | Keywords: | |
Cc: |
Description
New point version.
Change History (7)
comment:1 by , 14 months ago
comment:2 by , 14 months ago
When updating these packages, please fix the Title to capitalize Loader and Headers to match both the tarball name and the extracted directory.
comment:3 by , 14 months ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:5 by , 14 months ago
Vulkan-Headers has just one commit, but it does update a lot of header files and adds some new functions. The commit was titled "Update for Vulkan-Docs 1.3.273"
comment:6 by , 14 months ago
Vulkan-Loader:
Specify which layer type is being searched in log Makes it clear which layer type is being searched for in any given set of paths. Support Clang with GNU interface on Windows By using CMAKE_CXX_COMPILER_FRONTEND_VARIANT, CMake will now distinguish clang from clang-cl, allowing the correct command line argument formats to be used. Making tests work required fixing up windows_shim.cpp's usage of wcscpy since that was triggering deprecation warnings. A simple for loop was used instead. Add simple timer of ICD preloading in live tests Adds a simple executable that times how long vkEnumerateInstanceExtensionProperties takes over and over to see how well the ICD preloading functions. Unload drivers which report 0 physical devices The loader did not unload any ICD's which contained zero physical devices, which could cause premature exhaustion of memory in some circumstances, like 32 bit applications. While the policy of the loader has been to keep things open for the duration of the instance, these ICD's don't meaningfully participate in anything due to the lack of VkPhysicalDevices. This change adds a check after vkEnumeratePhysicalDevices where pPhysicalDevices is not NULL such that all loader_icd_terms which reported zero physical devices have its vkDestroyInstance called, and removed from the loader_instance's icd_term linked list. Use memset to clear structs Several places manually zero'd out each member of a struct. This makes it possible to forget to zero out members, especially as structs change over time. Using memset prevents that possibility. Fix spelling mistakes in comments These creep in over time, and make the codebase harder to understand due to spelling errors causing confusion when reading comments. build: Update to header 1.3.273
comment:7 by , 14 months ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Note:
See TracTickets
for help on using tickets.
By the way, I think there is a mistake in Vulkan-Loader dependencies. I'm pretty sure lxml is not required (I can build Vulkan-Loader w/o lxml), and I cannot find a reference to lxml in the code.