-
Notifications
You must be signed in to change notification settings - Fork 419
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
Minor debugging enhancements. #2317
Minor debugging enhancements. #2317
Conversation
Thanks for pushing this forward, I was out last week and didn't get to it! I definitely glossed over the label part in the original submission. |
The {
NSAutoreleasePool *pool = [[NSAutoreleasePool alloc] init];
// contents of block...
[pool release];
} For a long time creating an
Anyway, this all sounds really complicated, and I'm not sure it's worth it, and why did I write all this again? * Replacing a method's |
I was thinking of just using the pool for these label strings, and maybe some other selected allocations (not sure if there are any that would benefit from it). We could
|
- On GPU error, log indicate whether device has been resumed or lost, and report as either warning or error, respectively. - On VkDevice loss, stop device-scope GPU capture. - In debug mode, include address of Metal object in Metal object label. - Replace setLabelIfNotNil() with setMetalObjectLabel() and move inside MVKVulkanAPIObject to allow access to config info. - When logging contents of MVKDescriptorPool, only list descriptor types with allocations. - Add @autoreleasepool around complex NSString creation.
38a170b
to
69fbea0
Compare
The I've pushed an update that eliminates all of the Hopefully that covers what we need here, and we won't need to introduce anything more complicated. |
VkDevice
loss, stop device-scope GPU capture.setLabelIfNotNil()
withsetMetalObjectLabel()
and move insideMVKVulkanAPIObject
to allow access to config info.MVKDescriptorPool
, only list descriptor types with allocations.@autoreleasepool
around complexNSString
creation.Expansion and replacement of PR #2305.