Skip to content

Conversation

whitneywhtsang
Copy link
Contributor

@whitneywhtsang whitneywhtsang commented Sep 23, 2025

This PR also removes SPV_KHR_bfloat16 workaround.

Block by driver update.
CI with Agama 1188: https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/17932864080 (PASS)

Fixes #3914

@whitneywhtsang whitneywhtsang self-assigned this Sep 23, 2025
@@ -1 +1 @@
8e8c02c4b803062501a6bd36aecdbb041fa1daef
31c82e8cfc7cec7ab1cd8d3359fdd8781780c91c
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Need to go back to an older commit, as KhronosGroup/SPIRV-LLVM-Translator@8242eb7 makes some tests fail. Problem and reproducer is reported to the author.

Copy link
Contributor

Choose a reason for hiding this comment

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

Note: 31c82e8cfc7cec7ab1cd8d3359fdd8781780c91c is is the older commit.

@@ -1 +1 @@
8e8c02c4b803062501a6bd36aecdbb041fa1daef
31c82e8cfc7cec7ab1cd8d3359fdd8781780c91c
Copy link
Contributor

Choose a reason for hiding this comment

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

Note: 31c82e8cfc7cec7ab1cd8d3359fdd8781780c91c is is the older commit.


FetchContent_MakeAvailable(spirv-llvm-translator)

# FIXME: Don't apply patch when Agama driver is updated.
Copy link
Contributor

Choose a reason for hiding this comment

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

Removing old hack now that we have a new driver.

@etiotto etiotto marked this pull request as ready for review September 24, 2025 19:24
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.

[TEST] Fix bfloat16 test failures
2 participants