Skip to content

fix(ci): Remove C++17 flag for gfx90a in CK CI - #12417

Merged
illsilin merged 2 commits into
developfrom
users/jialuo/ck/remove_c17_flag
Sep 22, 2026
Merged

illsilin merged 2 commits into
developfrom
users/jialuo/ck/remove_c17_flag

Conversation

@JiaLuo-CAN

Copy link
Copy Markdown
Contributor

Motivation

Fix the pipeline failure of gfx90a caused by the use of C++20 feature std::type_identity in PR #11537.

Technical Details

Test Plan

Test Result

Submission Checklist

@JiaLuo-CAN
JiaLuo-CAN requested a review from a team as a code owner September 22, 2026 14:38
@therock-pr-bot

therock-pr-bot Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
📝 PR Description ✅ Pass —
⛔ Forbidden Files ✅ Pass —
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🔎 pre-commit ✅ Pass —
🚫 Draft PR 🔜 To Be Enabled —
🚩 Feature Flag 🔜 To Be Enabled —
📊 Code Coverage 🔜 To Be Enabled —
🤖 therock-pr-bot ✅ Pass —

🎉 All checks passed! This PR is ready for review.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

🙋 Wish to Override Policy?

@therock-pr-bot

Copy link
Copy Markdown

🎉 All checks passed! This PR is ready for review.

@illsilin
illsilin merged commit fc3e0f4 into develop Sep 22, 2026
114 of 117 checks passed
@illsilin
illsilin deleted the users/jialuo/ck/remove_c17_flag branch September 22, 2026 20:02
assistant-librarian Bot pushed a commit to ROCm/composable_kernel that referenced this pull request Sep 22, 2026
fix(ci): Remove C++17 flag for gfx90a in CK CI

## Motivation

Fix the pipeline failure of gfx90a caused by the use of C++20 feature
std::type_identity in PR #11537.

## Technical Details

<!-- Explain the changes along with any relevant GitHub links. -->

## Test Plan

<!-- Explain any relevant testing done to verify this PR. -->

## Test Result

<!-- Briefly summarize test outcomes. -->

## Submission Checklist

- [ ] Look over the contributing guidelines at
https://github.com/ROCm/TheRock/blob/main/GOVERNANCE.md#pull-requests.
@assistant-librarian

Copy link
Copy Markdown
Contributor

TheRock Submodule Bump Activity

Newest first

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.

3 participants