Skip to content

Conversation

@BrianHarrisonAMD
Copy link
Contributor

@BrianHarrisonAMD BrianHarrisonAMD commented Jan 6, 2026

Motivation

Move and rename MIOpen plugin to match build normalization document from TheRock (RFC0003).

Added a symlink temporarily to keep CI working as we transition this project.
This work will require updating the submodule in TheRock, swapping the build & install over to the new location instead, and then removing the symlink once everything has propagated between repos.

Related to #2316 (needs a few changes to be considered done though).

Technical Details

  • Move projects/hipdnn/plugins/miopen_legacy_plugin -> dnn_providers/miopen-provider
  • Add project files (.clang-format, .clang-tidy, .clinerules, .gitattributes, .gitignore) to match hipDNN
  • Add temporary symlink of dnn_providers/miopen-provider -> projects/hipdnn/plugins/miopen_legacy_plugin

Test Plan

Tests run and build same as before.
Standalone miopen-provider builds and tests run same as before.

Test Result

Passing build & tests.

@BrianHarrisonAMD BrianHarrisonAMD self-assigned this Jan 6, 2026
@BrianHarrisonAMD BrianHarrisonAMD requested review from a team as code owners January 6, 2026 23:07
@BrianHarrisonAMD BrianHarrisonAMD changed the title Move MIOpen plugin to match build normalization RFC from TheRock [hipDNN] Move MIOpen plugin to match build normalization RFC from TheRock Jan 6, 2026
Copy link
Contributor

@CMiservaAMD CMiservaAMD left a comment

Choose a reason for hiding this comment

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

Hopefully the symlink works.

@SamuelReeder SamuelReeder requested a review from a team as a code owner January 7, 2026 21:58
@SamuelReeder SamuelReeder merged commit c0435c6 into develop Jan 8, 2026
19 checks passed
@SamuelReeder SamuelReeder deleted the users/bharriso/relocate-miopen-plugin branch January 8, 2026 20:22
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.

7 participants