Skip to content

[master] Fix fused softmax ut#237

Merged
amd-sriram merged 1 commit intomasterfrom
fix_fused_softmax_ut
Jul 9, 2025
Merged

[master] Fix fused softmax ut#237
amd-sriram merged 1 commit intomasterfrom
fix_fused_softmax_ut

Conversation

@amd-sriram
Copy link
Copy Markdown
Collaborator

@amd-sriram amd-sriram commented Jul 9, 2025

Replacing c10_warp_size and platform based warp_size values with at::cuda::warp_size() values

Fixes : https://ontrack-internal.amd.com/browse/SWDEV-541725

Cherry-picked to release/1.5.0 branch via #242

Cherry-picked to release/1.6.0 branch via #243

Cherry-picked to release/1.7.0 branch via #244

Cherry-picked to release/1.4.0 branch via #246

@amd-sriram
Copy link
Copy Markdown
Collaborator Author

! cherry-pick --onto release/1.5.0 release/1.6.0 release/1.7.0

@okakarpa
Copy link
Copy Markdown
Collaborator

okakarpa commented Jul 9, 2025

amd-sriram added a commit that referenced this pull request Jul 9, 2025
…rp_size() (#237) (#242)

Co-authored-by: Sriram Kumar <skishore@amd.com>
amd-sriram added a commit that referenced this pull request Jul 9, 2025
…rp_size() (#237) (#244)

Co-authored-by: Sriram Kumar <skishore@amd.com>
amd-sriram added a commit that referenced this pull request Jul 9, 2025
…rp_size() (#237) (#243)

Co-authored-by: Sriram Kumar <skishore@amd.com>
@amd-sriram
Copy link
Copy Markdown
Collaborator Author

! cherry-pick --onto release/1.4.0

@okakarpa
Copy link
Copy Markdown
Collaborator

Created branch autogenerated/release/1.4.0_cherry-pick_pr-237 and #246. It contains a merge conflict. Please resolve it

jithunnair-amd pushed a commit to ROCm/pytorch that referenced this pull request Jul 11, 2025
Apex extensions import test - ROCm/apex#245

Fixing the C10_warpsize issue. replacing the macros with
at::cuda::warp_size() - ROCm/apex#237


Fixes : https://ontrack-internal.amd.com/browse/SWDEV-541725
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