Skip to content

revert(ck): magic division for long_index_t (#8983) - #9110

Merged
marbre merged 1 commit into
release/therock-7.14from
users/jlichtne/ROCM-27360-revert-magic-division-cherry-pick
Jul 7, 2026
Merged

marbre merged 1 commit into
release/therock-7.14from
users/jlichtne/ROCM-27360-revert-magic-division-cherry-pick

Conversation

@JonathanLichtnerAMD

@JonathanLichtnerAMD JonathanLichtnerAMD commented Jul 6, 2026 •

Copy link
Copy Markdown
Contributor

Cherry-picks commit 80c884a into the 7.14 release branch.


This reverts commit c25787b.

Motivation

Memfaults problems after this implementation.

Technical Details

revert magic division for large tensors. Return to standard division path

Test Plan

test_grouped_convnd_bwd_data

Test Result

passed

Submission Checklist

JIRA ID : ROCM-27282
JIRA ID : ROCM-27360

This reverts commit c25787b.

## Motivation

Memfaults problems after this implementation.

## Technical Details

revert magic division for large tensors. Return to standard division
path

## Test Plan

test_grouped_convnd_bwd_data

## Test Result

passed

## Submission Checklist

- [x] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.

JIRA ID : ROCM-27282
JIRA ID : ROCM-27360

@marbre marbre left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approving for devops, assuming CI passes.

@marbre
marbre merged commit a6950e8 into release/therock-7.14 Jul 7, 2026
37 of 48 checks passed
@marbre
marbre deleted the users/jlichtne/ROCM-27360-revert-magic-division-cherry-pick branch July 7, 2026 06:50
marbre added a commit to ROCm/TheRock that referenced this pull request Jul 7, 2026
Bumps rocm-libraries and rocm-systems to include the following
cherry-picks:

* ROCm/rocm-libraries#9105
* ROCm/rocm-libraries#9110
* ROCm/rocm-systems#8170
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants