Skip to content

Add hipFile examples#465

Open
j-stephan wants to merge 13 commits into
amd-stagingfrom
jstephan/hipFile
Open

Add hipFile examples#465
j-stephan wants to merge 13 commits into
amd-stagingfrom
jstephan/hipFile

Conversation

@j-stephan

Copy link
Copy Markdown
Contributor

Motivation

Adds examples for hipFile, a new rocm-systems component.

Technical Details

Supports all ROCm-capable GPUs, but only works on Linux.

Test Plan

Currently testing locally on Linux machines.

Test Result

Will update once results are obtained.

Added/Updated documentation?

  • Yes
  • No, does not apply to this PR.

Included Visual Studio files?

  • Yes
  • No, does not apply to this PR.

Submission Checklist

Signed-off-by: Jan Stephan <jan.stephan@amd.com>
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
@j-stephan j-stephan requested review from a team as code owners June 12, 2026 08:19
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
@j-stephan

Copy link
Copy Markdown
Contributor Author

hipFile is not yet part of TheRock. Need to wait for ROCm/TheRock#5217 and a follow-up PR before we can merge this.

@zichguan-amd zichguan-amd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Missing .gitignore

Comment thread Systems/Makefile Outdated
# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
# SOFTWARE.

SUBDIRECTORIES := hipFile

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

For this can we add a check similar to

# Only build libraries whose headers are present
ifneq ($(wildcard $(INCLUDE_DIR)/hiptensor/hiptensor.hpp),)
so Make can properly skip the build when hipFile is not available?

Signed-off-by: Jan Stephan <jan.stephan@amd.com>
derobins added a commit to ROCm/TheRock that referenced this pull request Jun 16, 2026
## Motivation

Add hipFile (from rocm-systems) to TheRock

## NOTE
This PR depends upon a third-party sysdeps dependency PRs:
* util-linux (for libmount): #5337

## Technical Details

* Add a new storage-libs stage/grouping
* Add hipFile to CLAUDE.md
* Add storage-libs to root CMakeLists.txt
* Add THEROCK_ENABLE_HIPFILE CMake option (default ON)
* Add THEROCK_ENABLE_STORAGE_LIBS CMake option (default ON) 
* Add storage-libs to build_tools/analyze_build_times.py
* Add storage-libs to bump list in build_tools/bump_submodules.py
* Add storage-libs and hipfile info to:
  * docs/development/build_system.md
  * docs/development/windows_support.md
* Add new file: storage-libs/CMakeLists.txt
* Add new file: storage-libs/artifact-hipfile.toml

Packaging will be added in a follow-up PR

This PR needed for rocm-examples'
ROCm/rocm-examples#465

## Jira ID

ROCM-1261
AIHIPFILE-170

## Test Plan

Verify TheRock CI passes

## Test Result

TBD

## Submission Checklist

- [x] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
amd-chiranjeevi pushed a commit to ROCm/TheRock that referenced this pull request Jun 17, 2026
## Motivation

Add hipFile (from rocm-systems) to TheRock

## NOTE
This PR depends upon a third-party sysdeps dependency PRs:
* util-linux (for libmount): #5337

## Technical Details

* Add a new storage-libs stage/grouping
* Add hipFile to CLAUDE.md
* Add storage-libs to root CMakeLists.txt
* Add THEROCK_ENABLE_HIPFILE CMake option (default ON)
* Add THEROCK_ENABLE_STORAGE_LIBS CMake option (default ON) 
* Add storage-libs to build_tools/analyze_build_times.py
* Add storage-libs to bump list in build_tools/bump_submodules.py
* Add storage-libs and hipfile info to:
  * docs/development/build_system.md
  * docs/development/windows_support.md
* Add new file: storage-libs/CMakeLists.txt
* Add new file: storage-libs/artifact-hipfile.toml

Packaging will be added in a follow-up PR

This PR needed for rocm-examples'
ROCm/rocm-examples#465

## Jira ID

ROCM-1261
AIHIPFILE-170

## Test Plan

Verify TheRock CI passes

## Test Result

TBD

## Submission Checklist

- [x] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
Signed-off-by: Jan Stephan <jan.stephan@amd.com>
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.

2 participants