Skip to content

test: surface compute sanitizer logs#4752

Draft
WilliamK112 wants to merge 1 commit into
NVIDIA:mainfrom
WilliamK112:codex/surface-sanitizer-logs
Draft

test: surface compute sanitizer logs#4752
WilliamK112 wants to merge 1 commit into
NVIDIA:mainfrom
WilliamK112:codex/surface-sanitizer-logs

Conversation

@WilliamK112

Copy link
Copy Markdown

Summary

  • print fresh Compute Sanitizer log contents to stderr when the sanitizer wrapper exits nonzero
  • preserve the original sanitizer exit status and existing sanitizer_for_pid_*.log files
  • keep success-path output quiet

Closes #1337.

Validation

  • git diff --check
  • bash -n build/sanitizer-java/bin/java
  • ran the wrapper with a fake failing compute-sanitizer; verified stderr includes the simulated sanitizer report and the wrapper exits with the sanitizer status
  • ran the wrapper with a fake successful compute-sanitizer; verified stderr remains empty and the wrapper exits 0

Signed-off-by: WilliamK112 <164879897+WilliamK112@users.noreply.github.com>
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.

Make the error log more friendly when Compute Sanitizer fails the build.

2 participants