Skip to content

Conversation

@etiotto
Copy link
Contributor

@etiotto etiotto commented Dec 1, 2025

No description provided.

@etiotto etiotto self-assigned this Dec 1, 2025
@etiotto etiotto changed the title TEST Adjust unit test after upstream AxisAnalysi changes. Dec 2, 2025
@etiotto etiotto requested review from anmyachev, chengjunlu and whitneywhtsang and removed request for whitneywhtsang December 2, 2025 17:35
Copy link
Contributor

@anmyachev anmyachev left a comment

Choose a reason for hiding this comment

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

LGTM. I wonder if this change affects performance?

@etiotto etiotto changed the title Adjust unit test after upstream AxisAnalysi changes. Adjust unit test after upstream AxisAnalysis changes. Dec 2, 2025
@etiotto
Copy link
Contributor Author

etiotto commented Dec 2, 2025

LGTM. I wonder if this change affects performance?

Benchmark run: https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/19872998954

@etiotto
Copy link
Contributor Author

etiotto commented Dec 3, 2025

The benchmark run I kicked off above showed a degradation in flex attention for one shape (144 TFlops -> 131 TFlops):

image

However I rerun the benchmark one more time (https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/19902682824/job/57050650262) and got:

image

Also the degradation was not reproducible locally on a PVC1100 machine. So false alarm.

@etiotto etiotto marked this pull request as ready for review December 3, 2025 18:32
@etiotto etiotto linked an issue Dec 3, 2025 that may be closed by this pull request
@whitneywhtsang whitneywhtsang merged commit 0e5643b into main Dec 4, 2025
25 checks passed
@whitneywhtsang whitneywhtsang deleted the etiotto.fix_5588 branch December 4, 2025 14:18
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.

Reland upstream commit 31281bc

5 participants