Skip to content

Test: bump rocm-libraries to fix_warnings (-Werror validation)#5166

Draft
BradPepersAMD wants to merge 3 commits into
mainfrom
users/brpepers/test_for_warnings
Draft

Test: bump rocm-libraries to fix_warnings (-Werror validation)#5166
BradPepersAMD wants to merge 3 commits into
mainfrom
users/brpepers/test_for_warnings

Conversation

@BradPepersAMD
Copy link
Copy Markdown
Contributor

Purpose

Validation-only PR — not intended to merge.

This bumps the rocm-libraries submodule to branch users/brpepers/fix_warnings (ROCm/rocm-libraries#7257), which replaces MIOpen's -Weverything with -Wall -Wextra -Werror.

The goal is to exercise the full GPU target matrix via ci:run-all-archs CI to confirm MIOpen is warning-free across all architectures before the rocm-libraries PR is merged.

What changed

  • rocm-libraries submodule: 89b3fc4c28e9c7e6c02aa (users/brpepers/fix_warnings)

Next steps

  1. Apply the ci:run-all-archs label to trigger all-architecture CI
  2. Monitor for any -Werror-related build failures
  3. If clean, merge Enable -Werror for MIOpen by replacing -Weverything with -Wall -Wextra rocm-libraries#7257 and close this PR without merging

🤖 Generated with Claude Code

…tion

Validation-only bump — not intended to merge. This exercises the full
GPU target matrix (via ci:run-all-archs) to confirm MIOpen -Werror is
clean across all architectures before merging ROCm/rocm-libraries#7257.

Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
@BradPepersAMD BradPepersAMD requested a review from a team as a code owner May 11, 2026 10:59
@BradPepersAMD BradPepersAMD added the ci:run-all-archs Opt-in to building for all architectures on a pull request label May 11, 2026
@BradPepersAMD BradPepersAMD marked this pull request as draft May 11, 2026 11:00
BradPepersAMD and others added 2 commits May 11, 2026 06:56
solver_nm is unused when MIOPEN_ENABLE_AI_KERNEL_TUNING and
MIOPEN_USE_COMPOSABLEKERNEL are not defined. Fixes build failure
on gfx900 and gfx906.

Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
…pp for non-CK builds

Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:run-all-archs Opt-in to building for all architectures on a pull request

Projects

Status: TODO

Development

Successfully merging this pull request may close these issues.

1 participant