Skip to content

split sanitizers into per-sanitizer features, flags, and constraints#5

Draft
RSingh1511 wants to merge 1 commit into
eclipse-score:mainfrom
RSingh1511:rs/sanitizer-split
Draft

split sanitizers into per-sanitizer features, flags, and constraints#5
RSingh1511 wants to merge 1 commit into
eclipse-score:mainfrom
RSingh1511:rs/sanitizer-split

Conversation

@RSingh1511
Copy link
Copy Markdown
Contributor

@RSingh1511 RSingh1511 commented May 22, 2026

Summary

Split sanitizers into independent, composable units — each sanitizer (ASan, UBSan, LSan, TSan) now has its own bool_flag, Bazel feature, and execution constraint
Enables enabling any single sanitizer in isolation (--config=asan, --config=tsan, etc.)
Supports selective composition (--config=asan_ubsan_lsan)
Adds fine-grained constraints (no_asan, only_tsan, etc.) to skip tests incompatible with a specific sanitizer

Notes for Reviewer

Pre-Review Checklist for the PR Author

  • PR title is short, expressive and meaningful
  • Commits are properly organized
  • Relevant issues are linked in the References section
  • Tests are conducted
  • Unit tests are added

Checklist for the PR Reviewer

  • Commits are properly organized and messages follow S-CORE commit guidelines
  • Unit tests have been written for new behavior
  • Public API is documented
  • PR title describes the changes

Post-review Checklist for the PR Author

  • All open points are addressed and tracked via issues

References

Closes #4

@RSingh1511 RSingh1511 force-pushed the rs/sanitizer-split branch from 1a93c35 to 953168c Compare May 22, 2026 13:16
@RSingh1511 RSingh1511 requested a review from rmaddikery May 22, 2026 13:38
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.

Separation of santitizer config to align with current repos

1 participant