Skip to content

Conversation

@derek-gerstmann
Copy link
Contributor

@derek-gerstmann derek-gerstmann commented Oct 29, 2024

  • Fix ArrayStride for arrays of vectors bound to buffer blocks.
  • Make a local copy of allocators to avoid dangling references during destruction
  • Cleanup Vulkan Memory Allocator destruction to avoid duplicate calls to destroy block allocator

Fixes #8291
Fixes #8287
Fixes #8293

Derek Gerstmann added 3 commits October 28, 2024 13:48
This resolves issues for CodeGen and runtime errors for bilateral_grid.
destruction calls for block allocator.
@derek-gerstmann derek-gerstmann self-assigned this Oct 29, 2024
@derek-gerstmann derek-gerstmann marked this pull request as draft October 29, 2024 00:02
@halide halide deleted a comment from derek-gerstmann Oct 29, 2024
@derek-gerstmann derek-gerstmann marked this pull request as ready for review October 30, 2024 18:29
@derek-gerstmann
Copy link
Contributor Author

Okay, this is good to go. I've been testing this on my local machine configs and this passes cleanly on Linux Ubuntu 22.04 w/Vulkan SDK v1.3.296 w/NVIDIA GeForce RTX 3070 TI.

@derek-gerstmann
Copy link
Contributor Author

I'm also testing changes to avoid over-use of descriptor sets on resource constrained devices, but that'll be a larger PR.

@abadams
Copy link
Member

abadams commented Oct 30, 2024

Failure is unrelated flake

@abadams abadams merged commit 4052dda into main Oct 30, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

4 participants