Skip to content

Add missing sanitizer interface include#24263

Merged
asmorkalov merged 1 commit intoopencv:4.xfrom
georgthegreat:msan-include
Sep 13, 2023
Merged

Add missing sanitizer interface include#24263
asmorkalov merged 1 commit intoopencv:4.xfrom
georgthegreat:msan-include

Conversation

@georgthegreat
Copy link
Copy Markdown
Contributor

@georgthegreat georgthegreat commented Sep 13, 2023

Fixes a bunch of the following errors:

error: use of undeclared identifier '__msan_unpoison'
      CV_ANNOTATE_MEMORY_IS_INITIALIZED(w, sizeof(fptype) * std::min(m, n));

Copy link
Copy Markdown
Contributor

@asmorkalov asmorkalov left a comment

Choose a reason for hiding this comment

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

👍

@opencv-alalek
Copy link
Copy Markdown
Contributor

/cc @vrabaud

@vrabaud
Copy link
Copy Markdown
Contributor

vrabaud commented Sep 13, 2023

LGTM: I guess our internal tooling includes it by default. Thx!

@asmorkalov asmorkalov merged commit 4790a37 into opencv:4.x Sep 13, 2023
@georgthegreat georgthegreat deleted the msan-include branch September 13, 2023 14:40
@asmorkalov asmorkalov mentioned this pull request Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants