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] switch rocm build to clang #19660

Closed
wants to merge 3 commits into from

Conversation

alekstheod
Copy link
Contributor

@alekstheod alekstheod commented Nov 21, 2024

This PR switches the default rocm build to clang as the gcc config is broken at the moment.

.bazelrc Outdated Show resolved Hide resolved
copybara-service bot pushed a commit to google/tsl that referenced this pull request Nov 22, 2024
Imported from GitHub PR openxla/xla#19660

This PR switches the default rocm build to clang as the gcc config is broken at the moment.

Copybara import of the project:

--
ea48f7c480d110eab3f133ed6ea8989da0e1e724 by Alexandros Theodoridis <[email protected]>:

[ROCm] switch rocm build to clang

--
2743fabafd6a358c05e858781064e7fa2e389c78 by Alexandros Theodoridis <[email protected]>:

Remove explicit clang path from the bazelrc rocm config

--
202dea0a80602cafdbee6067d8f20dc3055c6bbb by Alexandros Theodoridis <[email protected]>:

Address review comments

Merging this change closes #19660

FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#19660 from ROCm:ci_switch_rocm_build_to_clang 202dea0a80602cafdbee6067d8f20dc3055c6bbb
PiperOrigin-RevId: 699196393
copybara-service bot pushed a commit that referenced this pull request Nov 22, 2024
Imported from GitHub PR #19660

This PR switches the default rocm build to clang as the gcc config is broken at the moment.

Copybara import of the project:

--
ea48f7c by Alexandros Theodoridis <[email protected]>:

[ROCm] switch rocm build to clang

--
2743fab by Alexandros Theodoridis <[email protected]>:

Remove explicit clang path from the bazelrc rocm config

--
202dea0 by Alexandros Theodoridis <[email protected]>:

Address review comments

Merging this change closes #19660

FUTURE_COPYBARA_INTEGRATE_REVIEW=#19660 from ROCm:ci_switch_rocm_build_to_clang 202dea0
PiperOrigin-RevId: 699196393
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Nov 22, 2024
Imported from GitHub PR openxla/xla#19660

This PR switches the default rocm build to clang as the gcc config is broken at the moment.

Copybara import of the project:

--
ea48f7c480d110eab3f133ed6ea8989da0e1e724 by Alexandros Theodoridis <[email protected]>:

[ROCm] switch rocm build to clang

--
2743fabafd6a358c05e858781064e7fa2e389c78 by Alexandros Theodoridis <[email protected]>:

Remove explicit clang path from the bazelrc rocm config

--
202dea0a80602cafdbee6067d8f20dc3055c6bbb by Alexandros Theodoridis <[email protected]>:

Address review comments

Merging this change closes #19660

FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#19660 from ROCm:ci_switch_rocm_build_to_clang 202dea0a80602cafdbee6067d8f20dc3055c6bbb
PiperOrigin-RevId: 699196393
copybara-service bot pushed a commit that referenced this pull request Nov 22, 2024
Imported from GitHub PR #19660

This PR switches the default rocm build to clang as the gcc config is broken at the moment.

Copybara import of the project:

--
ea48f7c by Alexandros Theodoridis <[email protected]>:

[ROCm] switch rocm build to clang

--
2743fab by Alexandros Theodoridis <[email protected]>:

Remove explicit clang path from the bazelrc rocm config

--
202dea0 by Alexandros Theodoridis <[email protected]>:

Address review comments

Merging this change closes #19660

FUTURE_COPYBARA_INTEGRATE_REVIEW=#19660 from ROCm:ci_switch_rocm_build_to_clang 202dea0
PiperOrigin-RevId: 699196393
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Nov 22, 2024
Imported from GitHub PR openxla/xla#19660

This PR switches the default rocm build to clang as the gcc config is broken at the moment.

Copybara import of the project:

--
ea48f7c480d110eab3f133ed6ea8989da0e1e724 by Alexandros Theodoridis <[email protected]>:

[ROCm] switch rocm build to clang

--
2743fabafd6a358c05e858781064e7fa2e389c78 by Alexandros Theodoridis <[email protected]>:

Remove explicit clang path from the bazelrc rocm config

--
202dea0a80602cafdbee6067d8f20dc3055c6bbb by Alexandros Theodoridis <[email protected]>:

Address review comments

Merging this change closes #19660

FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#19660 from ROCm:ci_switch_rocm_build_to_clang 202dea0a80602cafdbee6067d8f20dc3055c6bbb
PiperOrigin-RevId: 699196393
copybara-service bot pushed a commit to google/tsl that referenced this pull request Nov 22, 2024
Imported from GitHub PR openxla/xla#19660

This PR switches the default rocm build to clang as the gcc config is broken at the moment.

Copybara import of the project:

--
ea48f7c480d110eab3f133ed6ea8989da0e1e724 by Alexandros Theodoridis <[email protected]>:

[ROCm] switch rocm build to clang

--
2743fabafd6a358c05e858781064e7fa2e389c78 by Alexandros Theodoridis <[email protected]>:

Remove explicit clang path from the bazelrc rocm config

--
202dea0a80602cafdbee6067d8f20dc3055c6bbb by Alexandros Theodoridis <[email protected]>:

Address review comments

Merging this change closes #19660

PiperOrigin-RevId: 699222609
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Nov 22, 2024
Imported from GitHub PR openxla/xla#19660

This PR switches the default rocm build to clang as the gcc config is broken at the moment.

Copybara import of the project:

--
ea48f7c480d110eab3f133ed6ea8989da0e1e724 by Alexandros Theodoridis <[email protected]>:

[ROCm] switch rocm build to clang

--
2743fabafd6a358c05e858781064e7fa2e389c78 by Alexandros Theodoridis <[email protected]>:

Remove explicit clang path from the bazelrc rocm config

--
202dea0a80602cafdbee6067d8f20dc3055c6bbb by Alexandros Theodoridis <[email protected]>:

Address review comments

Merging this change closes #19660

PiperOrigin-RevId: 699222609
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