Skip to content

Conversation

@yingcong-wu
Copy link
Contributor

It's better that we have an such option when we need to disable the leak detection.

@yingcong-wu yingcong-wu requested a review from a team as a code owner October 28, 2024 06:36
@github-actions github-actions bot added loader Loader related feature/bug sanitizer Sanitizer layer issues/changes/specification labels Oct 28, 2024
@yingcong-wu yingcong-wu marked this pull request as draft October 28, 2024 07:01
@yingcong-wu yingcong-wu marked this pull request as ready for review October 28, 2024 07:05
std::shared_ptr<DeviceInfo> &DeviceInfo);

private:
AsanOptions m_Options;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unnecessary changes

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a needed change because we need to make sure AsanOptions destucts after ContextInfo so that we can use it in the ~ContextInfo().

@yingcong-wu yingcong-wu marked this pull request as draft November 21, 2024 07:22
@yingcong-wu
Copy link
Contributor Author

Convert this to draft in order to fast-track #2232 .

@yingcong-wu yingcong-wu requested a review from pbalcer November 25, 2024 03:29
@yingcong-wu yingcong-wu marked this pull request as ready for review November 25, 2024 03:29
@yingcong-wu
Copy link
Contributor Author

CI fails with apt package installation. Could we restart the CI?

@pbalcer pbalcer added the ready to merge Added to PR's which are ready to merge label Nov 25, 2024
@kbenzie kbenzie added the v0.11.x Include in the v0.11.x release label Nov 25, 2024
@kbenzie kbenzie merged commit 0b5d8f9 into oneapi-src:main Dec 3, 2024
76 of 78 checks passed
@yingcong-wu yingcong-wu deleted the yc-PR/1028-leak-check-option branch January 3, 2025 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

loader Loader related feature/bug ready to merge Added to PR's which are ready to merge sanitizer Sanitizer layer issues/changes/specification v0.11.x Include in the v0.11.x release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants