Skip to content

Reduce differences to upstream in bf16 vs f16 GlobalISel code#880

Merged
jorickert merged 3 commits intoaie-publicfrom
jrickert.closer_to_upstream
Mar 31, 2026
Merged

Reduce differences to upstream in bf16 vs f16 GlobalISel code#880
jorickert merged 3 commits intoaie-publicfrom
jrickert.closer_to_upstream

Conversation

@jorickert
Copy link
Copy Markdown
Contributor

@jorickert jorickert commented Mar 27, 2026

This should make it easier to pull in upstream changes .

Instead of globally disabling Bf16 patterns for the GIsel, still allow them for selected targets

targetSupportsBF16Type is based on upstream commit 55ab63321804af36aa43cd1e8696c30285f0edc0
Base automatically changed from bump_to_d3446240 to aie-public March 27, 2026 14:05
@jorickert jorickert marked this pull request as ready for review March 27, 2026 14:06
@jorickert jorickert requested a review from isoard-amd March 27, 2026 14:06
@jorickert jorickert changed the title Make bf16 vs f16 in GlobalISel work more similar to upstream Reduce differences to upstream in bf16 vs f16 GlobalISel code Mar 27, 2026
@jorickert jorickert closed this Mar 27, 2026
@jorickert jorickert reopened this Mar 27, 2026
Copy link
Copy Markdown
Collaborator

@isoard-amd isoard-amd left a comment

Choose a reason for hiding this comment

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

Good. This align with [SPIRV] Enable bfloat16 arithmetic from upstream (llvm/llvm-project#166031)

@jorickert jorickert enabled auto-merge March 30, 2026 13:21
@jorickert jorickert disabled auto-merge March 30, 2026 13:34
@jorickert jorickert merged commit b718756 into aie-public Mar 31, 2026
7 checks passed
@jorickert jorickert deleted the jrickert.closer_to_upstream branch March 31, 2026 06:34
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.

3 participants