Fix minor performance issue with leak checker destroy by nrspruit · Pull Request #401 · oneapi-src/level-zero
Navigation Menu
{{ message }}
oneapi-src / level-zero Public
- Notifications You must be signed in to change notification settings
- Fork 127
Merged
nrspruit merged 1 commit intooneapi-src:masterfrom
Dec 11, 2025Merged
Fix minor performance issue with leak checker destroy#401
nrspruit merged 1 commit intooneapi-src:masterfrom
Fix minor performance issue with leak checker destroy#401
nrspruit merged 1 commit intooneapi-src:masterfrom
Conversation
Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
nrspruit
requested review from
Copilot and
rwmcguir
Copilot AI reviewed Dec 11, 2025
Copy link Copy Markdown
Copilot
AI
left a comment
Copilot AI left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR addresses a minor performance optimization in the leak checker's destroy functionality by reducing unnecessary string copies during validation.
Key Changes:
- Modified variable declaration to use const reference instead of value copy in the leak checker loop
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
rwmcguir approved these changes Dec 11, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment