Skip to content

Add support for riscv64 linux#25236

Closed
kxxt wants to merge 1 commit intobazelbuild:masterfrom
kxxt:riscv
Closed

Add support for riscv64 linux#25236
kxxt wants to merge 1 commit intobazelbuild:masterfrom
kxxt:riscv

Conversation

@kxxt
Copy link
Contributor

@kxxt kxxt commented Feb 8, 2025

This patch fixes bootstrapping bazel on riscv64 linux

In src/minimize_jdk.sh,

The usage of uname -p isn't portable and it returns unknown for me on riscv64.
So I changed it to uname -m.

@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Feb 8, 2025
kxxt added a commit to kxxt/archriscv-packages that referenced this pull request Feb 8, 2025
- Patch bazel to bootstrap it on riscv64. Upstream PR: bazelbuild/bazel#25236
- Force bazel to run with system JDK
@kxxt kxxt force-pushed the riscv branch 2 times, most recently from 4bf770f to 6a01125 Compare February 8, 2025 07:51
This patch fixes bootstrapping bazel on riscv64 linux
felixonmars pushed a commit to felixonmars/archriscv-packages that referenced this pull request Feb 8, 2025
- Patch bazel to bootstrap it on riscv64. Upstream PR: bazelbuild/bazel#25236
- Force bazel to run with system JDK
@sgowroji sgowroji added the team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website label Feb 10, 2025
Copy link
Member

@meteorcloudy meteorcloudy left a comment

Choose a reason for hiding this comment

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

Thanks!

@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Feb 10, 2025
@github-actions github-actions bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Feb 10, 2025
kxxt added a commit to kxxt/bazel that referenced this pull request Mar 29, 2025
This patch fixes bootstrapping bazel on riscv64 linux

In `src/minimize_jdk.sh`,

The usage of `uname -p` isn't portable and it returns `unknown` for me on riscv64.
So I changed it to `uname -m`.

Closes bazelbuild#25236.

PiperOrigin-RevId: 725251883
Change-Id: I5ddf82840a958137ddba4a521f4ee8502185d78d
kxxt added a commit to kxxt/bazel that referenced this pull request Mar 29, 2025
This patch fixes bootstrapping bazel on riscv64 linux

In `src/minimize_jdk.sh`,

The usage of `uname -p` isn't portable and it returns `unknown` for me on riscv64.
So I changed it to `uname -m`.

Closes bazelbuild#25236.

PiperOrigin-RevId: 725251883
Change-Id: I5ddf82840a958137ddba4a521f4ee8502185d78d
@iancha1992
Copy link
Member

@bazel-io fork 8.2.0

bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Mar 31, 2025
This patch fixes bootstrapping bazel on riscv64 linux

In `src/minimize_jdk.sh`,

The usage of `uname -p` isn't portable and it returns `unknown` for me on riscv64.
So I changed it to `uname -m`.

Closes bazelbuild#25236.

PiperOrigin-RevId: 725251883
Change-Id: I5ddf82840a958137ddba4a521f4ee8502185d78d
github-merge-queue bot pushed a commit that referenced this pull request Apr 1, 2025
This patch fixes bootstrapping bazel on riscv64 linux

In `src/minimize_jdk.sh`,

The usage of `uname -p` isn't portable and it returns `unknown` for me
on riscv64.
So I changed it to `uname -m`.

Closes #25236.

PiperOrigin-RevId: 725251883
Change-Id: I5ddf82840a958137ddba4a521f4ee8502185d78d

Commit
5f74b9a

Co-authored-by: kxxt <[email protected]>
@iancha1992
Copy link
Member

The changes in this PR have been included in Bazel 8.2.0 RC3. Please test out the release candidate and report any issues as soon as possible.
If you're using Bazelisk, you can point to the latest RC by setting USE_BAZEL_VERSION=8.2.0rc3. Thanks!

@kxxt
Copy link
Contributor Author

kxxt commented Apr 12, 2025

The changes in this PR have been included in Bazel 8.2.0 RC3. Please test out the release candidate and report any issues as soon as possible. If you're using Bazelisk, you can point to the latest RC by setting USE_BAZEL_VERSION=8.2.0rc3. Thanks!

8.2.0rc3 builds fine on riscv64. Thanks for the notification!

@kxxt kxxt deleted the riscv branch April 28, 2025 13:48
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 3, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 825073762
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 3, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 825073762
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 3, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 825073762
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 3, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 825073762
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 3, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 825073762
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 4, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 825073762
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 4, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 825073762
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 4, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 825073762
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 4, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 825073762
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 26, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 836231774
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 26, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 836231774
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Nov 26, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 836231774
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Dec 2, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 836231774
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Dec 2, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 836231774
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Dec 2, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 836231774
copybara-service bot pushed a commit to openxla/xla that referenced this pull request Dec 2, 2025
Imported from GitHub PR tensorflow/tensorflow#102607

This commit fixes #102159 which prevented TensorFlow from compiling on RISC-V 64 due to missing codepath in the build system.

RFC: Currently this patch overrides timeout of rules_python for all platforms which is obviously not ideal. How do we set pip package timeout here in Tensorflow?

See also:
- bazelbuild/bazel#25236
- bazelbuild/bazel#25699
- bazel-contrib/rules_python#3350
- #32812
- tensorflow/runtime#135
Copybara import of the project:

--
13b7db883e6ea0aa631490fb5f806c55ceea7cb5 by gns <[email protected]>:

[XLA:CPU] Add support for riscv64

Co-authored-by: Levi Zim <[email protected]>

--
a00c3a3ba94664798960cbc2fb0a6746fe0c3ac1 by gns <[email protected]>:

Add missing llvm:: namespace qualifiers for DTensor

Add the `llvm::` namespace prefix to `cast` and `isa`
where it was missing in the DTensor MLIR code.

--
febc5abc56ab302f5f39fe20176ed00c0c95cb7e by gns <[email protected]>:

runtime: add missing stdint header

Co-authored-by: Levi Zim <[email protected]>

--
6bf63fec56a6d961b06d932fe69b76eae89d8cb7 by gns <[email protected]>:

Add riscv64 support patch for rules_python

bazel-contrib/rules_python#3350

--
68c44fa9613d5577f36c51a192c932795e13ed83 by gns <[email protected]>:

tools: py: add riscv64 to pip and manylinux compilance test

--
3f2fe1e0256476449f64658219d997727c9cc534 by gns <[email protected]>:

Lift timeout of rules_python for BFS wheels

--
252a9b7ad2336b7abbb94e269b4a976b45d5d6e7 by gns <[email protected]>:

Revert llvm toolchain patch for riscv64

As already upstreamed.

Co-authored-by: Levi Zim <[email protected]>

--
51047a86131e991ffea74fccb8f55e3521237a64 by gns <[email protected]>:

Lift timeout for BFS wheels during pip init

--
3b8276420031969423c9a1f150341a0e41fa372e by gns <[email protected]>:

Refresh `rules_python` riscv64 patch

Co-authored-by: Levi Zim <[email protected]>

Merging this change closes #102607

FUTURE_COPYBARA_INTEGRATE_REVIEW=tensorflow/tensorflow#102607 from infiWang:riscv64 3b8276420031969423c9a1f150341a0e41fa372e
PiperOrigin-RevId: 836231774
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-OSS Issues for the Bazel OSS team: installation, release processBazel packaging, website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants