Skip to content

Conversation

@yingcong-wu
Copy link
Contributor

@yingcong-wu yingcong-wu commented Dec 5, 2024

Some extensions are not currently supported by device address sanitizer. We may add support for those later, but for now, we will report errors when using unsupported API to let users know instead of failing for other random errors and puzzling the users.

Intel/llvm PR: intel/llvm#16281

@yingcong-wu yingcong-wu requested a review from a team as a code owner December 5, 2024 09:30
@github-actions github-actions bot added the loader Loader related feature/bug label Dec 5, 2024
@yingcong-wu yingcong-wu changed the title [DevAsan] Report error when using unsupported extension API [DevAsan] Report error when uses unsupported extension API Dec 5, 2024
@yingcong-wu yingcong-wu changed the title [DevAsan] Report error when uses unsupported extension API [DevAsan] Report error when using unsupported extension API Dec 5, 2024
Copy link
Contributor

@AllanZyne AllanZyne left a comment

Choose a reason for hiding this comment

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

Cool!

@alycm alycm added the v0.11.x Include in the v0.11.x release label Dec 10, 2024
@pbalcer pbalcer added the ready to merge Added to PR's which are ready to merge label Dec 10, 2024
Some extensions are not currently supported by device address sanitizer.
We may add support for those later, but for now, we will report errors
when using unsupported API to let users know instead of failing for
other random errors and puzzling the users.
@kbenzie kbenzie force-pushed the yc-PR/1204-error-on-unsupported-feature branch from a9387f3 to b532350 Compare December 10, 2024 18:04
@kbenzie kbenzie merged commit 870f0ed into oneapi-src:main Dec 10, 2024
37 of 38 checks passed
sommerlukas pushed a commit to intel/llvm that referenced this pull request Dec 11, 2024
@yingcong-wu yingcong-wu deleted the yc-PR/1204-error-on-unsupported-feature 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 v0.11.x Include in the v0.11.x release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants