Skip to content

[CK] Work around staging compiler lifetime warning - #4419

Merged
illsilin merged 3 commits into
developfrom
amd/dev/janplehr/ck/workaround-compiler-lifetime-warnings
Feb 12, 2026
Merged

illsilin merged 3 commits into
developfrom
amd/dev/janplehr/ck/workaround-compiler-lifetime-warnings

Conversation

@jplehr

@jplehr jplehr commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

Motivation

The staging compiler enables lifetime-safety warnings and we already worked around a few of them.
This works around a few more instances that came up recently on gfx950 builds.
The initial PR that resolved most issues: ROCm/composable_kernel#3640

Technical Details

This follows the pattern to locally ignore the newly added lifetime-safety warnings that were moved from experimental to production in upstream LLVM.
As a result, CK turned them on and treats them as errors, which prevents the staging compiler from building CK.

Test Plan

Test Result

Submission Checklist

The staging compiler enabled lifetime-safety warnings and we already
worked around a few of them.

This works around a few more instances that came up recently.
@illsilin
illsilin enabled auto-merge (squash) February 10, 2026 18:17
@jplehr

jplehr commented Feb 12, 2026

Copy link
Copy Markdown
Contributor Author

Are the failures related to my changes / flake?

@illsilin

Copy link
Copy Markdown
Contributor

Are the failures related to my changes / flake?

no, i think the ci failures are not related to your changes. we currently have some flaky/broken tests in develop, cause our testing script was broken for a few days and some errors crept in.

@illsilin
illsilin merged commit e241f8b into develop Feb 12, 2026
18 checks passed
@illsilin
illsilin deleted the amd/dev/janplehr/ck/workaround-compiler-lifetime-warnings branch February 12, 2026 22:11
assistant-librarian Bot pushed a commit to ROCm/composable_kernel that referenced this pull request Feb 12, 2026
[CK] Work around staging compiler lifetime warning

## Motivation
The staging compiler enables lifetime-safety warnings and we already
worked around a few of them.
This works around a few more instances that came up recently on gfx950
builds.
The initial PR that resolved most issues:
#3640

## Technical Details
This follows the pattern to locally ignore the newly added
lifetime-safety warnings that were moved from experimental to production
in upstream LLVM.
As a result, CK turned them on and treats them as errors, which prevents
the staging compiler from building CK.

## Test Plan

## Test Result

## Submission Checklist

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

Co-authored-by: Illia Silin <98187287+illsilin@users.noreply.github.com>
kamuruga08 pushed a commit that referenced this pull request Feb 19, 2026
## Motivation
The staging compiler enables lifetime-safety warnings and we already
worked around a few of them.
This works around a few more instances that came up recently on gfx950
builds.
The initial PR that resolved most issues:
ROCm/composable_kernel#3640

## Technical Details
This follows the pattern to locally ignore the newly added
lifetime-safety warnings that were moved from experimental to production
in upstream LLVM.
As a result, CK turned them on and treats them as errors, which prevents
the staging compiler from building CK.

## Test Plan

## Test Result

## Submission Checklist

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

Co-authored-by: Illia Silin <98187287+illsilin@users.noreply.github.com>
kamuruga08 pushed a commit that referenced this pull request Feb 19, 2026
## Motivation
The staging compiler enables lifetime-safety warnings and we already
worked around a few of them.
This works around a few more instances that came up recently on gfx950
builds.
The initial PR that resolved most issues:
ROCm/composable_kernel#3640

## Technical Details
This follows the pattern to locally ignore the newly added
lifetime-safety warnings that were moved from experimental to production
in upstream LLVM.
As a result, CK turned them on and treats them as errors, which prevents
the staging compiler from building CK.

## Test Plan

## Test Result

## Submission Checklist

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

Co-authored-by: Illia Silin <98187287+illsilin@users.noreply.github.com>
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.

2 participants