Skip to content

Prefer ROCm headers of the HIP compiler over a system HIP install - #3486

Merged
jajhall merged 1 commit into
ERGO-Code:latestfrom
amontoison:hotfix_mi350
Oct 28, 2026
Merged

jajhall merged 1 commit into
ERGO-Code:latestfrom
amontoison:hotfix_mi350

Conversation

@amontoison

@amontoison amontoison commented Oct 28, 2026 •

Copy link
Copy Markdown

Description

I was testing quickly HiGHS this afternoon on a recent MI350 GPU with a non-standard installation and I needed
to pass a specific compilation flag.
This PR makes the detection of HIP headers more robust.

Checklist

  • I have read the contributing guidelines
  • This PR targets the latest branch
  • Tests are passing
  • Documentation was updated where relevant
  • This PR is not primarily AI-generated (per the AI contributions policy in CONTRIBUTING.md)

@codecov

codecov Bot commented Oct 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.31%. Comparing base (95395c8) to head (b8f5f6e).
⚠️ Report is 37 commits behind head on latest.

Additional details and impacted files
@@            Coverage Diff             @@
##           latest    #3486      +/-   ##
==========================================
- Coverage   75.32%   75.31%   -0.01%     
==========================================
  Files         460      460              
  Lines      112769   112769              
  Branches    17856    17856              
==========================================
- Hits        84938    84934       -4     
- Misses      27543    27547       +4     
  Partials      288      288              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@galabovaa galabovaa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me!

@galabovaa

Copy link
Copy Markdown
Contributor

Thank you, @amontoison

@jajhall
jajhall merged commit cf69402 into ERGO-Code:latest Oct 28, 2026
395 checks passed
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.

3 participants