Skip to content

[comgr][hotswap] Prove finite indirect control flow - #16

Draft
harsh-amd wants to merge 1 commit into
users/harsh/hotswap-split-ref-04a-external-targetsfrom
users/harsh/hotswap-split-ref-04b-finite-fixedpoint
Draft

[comgr][hotswap] Prove finite indirect control flow#16
harsh-amd wants to merge 1 commit into
users/harsh/hotswap-split-ref-04a-external-targetsfrom
users/harsh/hotswap-split-ref-04b-finite-fixedpoint

Conversation

@harsh-amd

Copy link
Copy Markdown
Owner

Scope

Solve reusable PC-materialized calls, bounded returns, and closed symbol-less return regions to a fixed point while publishing a separate unbounded-indirect-entry signal.

Stack

One-commit clean reference split from ROCm#3598, stacked after bounded external targets. ROCm#3598 remains unchanged.

Validation provenance

The exact commit is an ancestor of corpus-tested ab3cdd6. Its historical integration stage had zero success-to-failure transitions and the final cumulative corpus was 2,685/2,685.

Before upstream promotion

  • this remains a substantial semantic proof; split internal implementation helpers further during current-staging extraction if requested
  • apply clang-format
  • run fail-closed units, independent oracle comparisons, focused lit, and full corpus

lamb-j pushed a commit to ROCm/llvm-project that referenced this pull request Jul 28, 2026
## Scope

Solve reusable calls, bounded returns, and closed symbol-less return
regions to a fail-closed fixed point.

## Clean review range

- Clean fork PR: harsh-amd#16
- Exact #3598 source commit(s): bbc93a5
- Depends on bounded external PC targets.

## Review status

Draft for early review. GitHub cannot base an upstream ROCm PR on a
branch in the fork, so the Files changed view is temporarily cumulative
with unmerged prerequisites. Review the clean fork PR and exact source
commits linked above. This branch will be rebased and reduced after
dependencies land; do not merge the cumulative snapshot.

#3598 remains unchanged as the immutable 2,685/2,685
integration reference.

## Validation provenance

The source commits are exact ancestors of corpus-tested ab3cdd6. The
cumulative integration passed 2,685/2,685 corpus paths, 213/213
HotswapMCTests, and COMGR lit with 168 passed, 14 unsupported, and 0
failed. Standalone current-staging formatting, focused tests, CI, and
the appropriate corpus transition gate are required before this draft
becomes ready.

---------

Co-authored-by: SourishW <87539392+SourishW@users.noreply.github.com>
lamb-j pushed a commit to ROCm/llvm-project that referenced this pull request Jul 28, 2026
## Scope

Solve reusable calls, bounded returns, and closed symbol-less return
regions to a fail-closed fixed point.

## Clean review range

- Clean fork PR: harsh-amd#16
- Exact #3598 source commit(s): bbc93a5
- Depends on bounded external PC targets.

## Review status

Draft for early review. GitHub cannot base an upstream ROCm PR on a
branch in the fork, so the Files changed view is temporarily cumulative
with unmerged prerequisites. Review the clean fork PR and exact source
commits linked above. This branch will be rebased and reduced after
dependencies land; do not merge the cumulative snapshot.

#3598 remains unchanged as the immutable 2,685/2,685
integration reference.

## Validation provenance

The source commits are exact ancestors of corpus-tested ab3cdd6. The
cumulative integration passed 2,685/2,685 corpus paths, 213/213
HotswapMCTests, and COMGR lit with 168 passed, 14 unsupported, and 0
failed. Standalone current-staging formatting, focused tests, CI, and
the appropriate corpus transition gate are required before this draft
becomes ready.

---------

Co-authored-by: SourishW <87539392+SourishW@users.noreply.github.com>
(cherry picked from commit b623f41)
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.

1 participant