Skip to content

Group PTX fast-math optimization passes under :fastmath#828

Merged
maleadt merged 2 commits into
mainfrom
fastm
Jun 3, 2026
Merged

Group PTX fast-math optimization passes under :fastmath#828
maleadt merged 2 commits into
mainfrom
fastm

Conversation

@wsmoses
Copy link
Copy Markdown
Member

@wsmoses wsmoses commented Jun 3, 2026

cc @gbaraldi @giordano

split out from #823

@wsmoses wsmoses requested review from maleadt and vchuravy June 3, 2026 16:15
Comment thread src/ptx.jl Outdated
@maleadt maleadt changed the title Conditionally disable ptx fastmath flags Group PTX fast-math optimization passes under :fastmath Jun 3, 2026
@maleadt
Copy link
Copy Markdown
Member

maleadt commented Jun 3, 2026

Otherwise LGTM

Co-authored-by: Tim Besard <tim.besard@gmail.com>
@maleadt maleadt merged commit 5d88d3c into main Jun 3, 2026
36 of 37 checks passed
@maleadt maleadt deleted the fastm branch June 3, 2026 19:45
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.30%. Comparing base (f42d1cf) to head (0f8a21f).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #828      +/-   ##
==========================================
- Coverage   79.31%   79.30%   -0.02%     
==========================================
  Files          25       25              
  Lines        4516     4517       +1     
==========================================
  Hits         3582     3582              
- Misses        934      935       +1     

☔ View full report in Codecov by Sentry.
📢 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.

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.

2 participants