Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ROCm] implement hermetic rocm dependency #64

Closed
wants to merge 2 commits into from

Conversation

alekstheod
Copy link

@alekstheod alekstheod commented Nov 12, 2024

This change has as a goal to introduce an external dependency to the rocm library and tools.

Building xla with the hermetic rocm is done by using these env variables:
--repo_env=OS=ubuntu_20.04 --repo_env=ROCM_VERSION=6.2.0
One has to provie OS version and ROCm version to initialize a proper rocm repository.
If these flags are not set then default ROCm installation will be used to build XLA.

Sample command:
bazel --output_base=./bazel-build-dir run --config=rocm //xla/... --repo_env=OS=ubuntu_20.04 --repo_env=ROCM_VERSION=6.2.0

@xla-rotation could you review my PR please?

@alekstheod alekstheod force-pushed the atheodor/implement_hermetic_rocm_dependency branch from f743561 to 4377c1c Compare November 13, 2024 12:37
@alekstheod alekstheod changed the title WIP implement hermetic rocm dependency implement hermetic rocm dependency Nov 13, 2024
@alekstheod alekstheod changed the title implement hermetic rocm dependency [ROCm] implement hermetic rocm dependency Nov 15, 2024
@alekstheod alekstheod changed the base branch from main to rocm-jaxlib-v0.4.30-qa November 18, 2024 13:09
@alekstheod alekstheod changed the base branch from rocm-jaxlib-v0.4.30-qa to rocm-jaxlib-v0.4.31-qa November 18, 2024 13:10
@alekstheod alekstheod changed the base branch from rocm-jaxlib-v0.4.31-qa to main November 18, 2024 13:11
@alekstheod alekstheod closed this Nov 18, 2024
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.

1 participant