Skip to content

Fix hipFile ASan runtime discovery for per-target layouts - #8719

Merged
sa-faizal merged 4 commits into
mainfrom
users/safaizal/hipfile-asan-runtime-20261003
Oct 8, 2026
Merged

sa-faizal merged 4 commits into
mainfrom
users/safaizal/hipfile-asan-runtime-20261003

Conversation

@sa-faizal

@sa-faizal sa-faizal commented Oct 3, 2026 •

Copy link
Copy Markdown
Contributor

hipFile host-ASan tests fail before execution because the launcher queries only libclang_rt.asan-x86_64.so, while per-target toolchains use libclang_rt.asan.so.

Query the existing clang++ location for the current filename, then the legacy filename, and validate the resolved file. The change is limited to the existing launcher; no configuration override or tests are added.

Validation:

  • On MI325 (gfx942), the reduced launcher passed 694/694 existing unit tests using host-ASan artifacts. ASAN_RUNTIME_PATH was unset and the ASan runtime directory was absent from LD_LIBRARY_PATH, directly exercising fallback discovery.
  • Resolver probes passed for both filenames and rejection of empty, echoed, or nonexistent paths. Syntax and repository Black checks passed.
  • System and stress suites were not run.

Original failure.

ISSUE ID : #6332

@therock-pr-bot

therock-pr-bot Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
📝 PR Description ✅ Pass —
⛔ Forbidden Files ✅ Pass —
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🔎 pre-commit ✅ Pass —
🚫 Draft PR 🔜 To Be Enabled —
🚩 Feature Flag 🔜 To Be Enabled —
📊 Code Coverage 🔜 To Be Enabled —
🤖 therock-pr-bot ✅ Pass —

🎉 All checks passed! This PR is ready for review.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

🙋 Wish to Override Policy?

@therock-pr-bot

therock-pr-bot Bot commented Oct 3, 2026

Copy link
Copy Markdown

🎉 All checks passed! This PR is ready for review.

@sa-faizal
sa-faizal marked this pull request as ready for review October 3, 2026 17:07
@sa-faizal
sa-faizal requested a review from akolliasAMD October 3, 2026 17:07
@sa-faizal
sa-faizal merged commit 5addcc6 into main Oct 8, 2026
172 checks passed
@sa-faizal
sa-faizal deleted the users/safaizal/hipfile-asan-runtime-20261003 branch October 8, 2026 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants