Skip to content

Metal: Align with Apple's compilers.#833

Merged
maleadt merged 2 commits into
mainfrom
tb/metal
Jun 5, 2026
Merged

Metal: Align with Apple's compilers.#833
maleadt merged 2 commits into
mainfrom
tb/metal

Conversation

@maleadt
Copy link
Copy Markdown
Member

@maleadt maleadt commented Jun 5, 2026

No description provided.

maleadt and others added 2 commits June 5, 2026 10:10
Apple's metal compiler emits the architecture as air64_v<major><minor>
(air64_v26 for AIR 2.6, etc.), and its tools derive the expected AIR
version from the triple, flagging a mismatch with the air.version module
metadata. Match that for AIR 2.6 and up; older toolchains used the plain
air64, so keep that for earlier targets.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Apple's metal compiler records the compile options it was invoked with in
an air.compile_options named metadata node. Emit the same: denorms_disable
and framebuffer_fetch_enable to match its defaults, and fast_math_enable or
fast_math_disable depending on target.fastmath, which already selects the
relaxed air.fast_* math device functions.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@maleadt maleadt merged commit a912d8e into main Jun 5, 2026
34 of 37 checks passed
@maleadt maleadt deleted the tb/metal branch June 5, 2026 08:55
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.41%. Comparing base (81a0987) to head (a04cfb7).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #833      +/-   ##
==========================================
+ Coverage   79.36%   79.41%   +0.05%     
==========================================
  Files          25       25              
  Lines        4584     4591       +7     
==========================================
+ Hits         3638     3646       +8     
+ Misses        946      945       -1     

☔ 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.

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.

1 participant