Skip to content

Turn off UMF_DEVELOPER_MODE in CI Sanitizers jobs - #1375

Merged
bratpiorka merged 1 commit into
oneapi-src:mainfrom
ldorau:Turn_off_UMF_DEVELOPER_MODE_in_CI_Sanitizers_jobs
Jun 13, 2025
Merged

Turn off UMF_DEVELOPER_MODE in CI Sanitizers jobs#1375
bratpiorka merged 1 commit into
oneapi-src:mainfrom
ldorau:Turn_off_UMF_DEVELOPER_MODE_in_CI_Sanitizers_jobs

Conversation

@ldorau

@ldorau ldorau commented Jun 12, 2025

Copy link
Copy Markdown
Contributor

Description

The UMF_DEVELOPER_MODE option turns on various additional debug checks that should be turned off in all Valgrind and Sanitizers builds, because we do not want to look for races in additional debug checks.

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly

The UMF_DEVELOPER_MODE option turns on various additional
debug checks that should be turned off in all Valgrind
and Sanitizers builds, because we do not want to look
for races in additional debug checks.

Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com>
@ldorau
ldorau marked this pull request as ready for review June 12, 2025 12:04
@ldorau
ldorau requested a review from a team as a code owner June 12, 2025 12:04
@lplewa

lplewa commented Jun 12, 2025

Copy link
Copy Markdown
Contributor

because we do not want to look for races in additional debug checks.

Are we? Imho If debug checks have bugs, if we enable them to debug something, we can start debug different issue.

@ldorau

ldorau commented Jun 12, 2025

Copy link
Copy Markdown
Contributor Author

because we do not want to look for races in additional debug checks.

Are we? Imho If debug checks have bugs, if we enable them to debug something, we can start debug different issue.

Yes, we do not want to check races between additional debug variables added by this option that do not exist in normal Release/Debug code when this option is turned off.
This option turns on checks that are very useful in all builds except of Valgrind and Sanitizer ones.

@ldorau

ldorau commented Jun 13, 2025

Copy link
Copy Markdown
Contributor Author

@bratpiorka please review

@bratpiorka
bratpiorka merged commit b1ca59d into oneapi-src:main Jun 13, 2025
88 checks passed
@ldorau
ldorau deleted the Turn_off_UMF_DEVELOPER_MODE_in_CI_Sanitizers_jobs branch June 16, 2025 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants