-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Invalid compiler_builtins
produced for loongarch64-unknown-none
#125908
Labels
A-CI
Area: Our Github Actions CI
C-bug
Category: This is a bug.
O-loongarch
Target: LoongArch (LA32R, LA32S, LA64)
T-bootstrap
Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Comments
bors
added a commit
to rust-lang-ci/rust
that referenced
this issue
Jun 30, 2024
Fix x86_64 code being produced for bare-metal LoongArch targets' `compiler_builtins` Formerly the `loongarch*-*-none*` targets were added to the `dist-various-2` CI job, but no corresponding toolchain was added along with them. This meant the `compiler_builtins` for the targets were built with the host toolchain. As the other `dist-various` toolchains are mostly pre-built so far, to avoid burdening them with crosstool-ng builds, simply move the two bare-metal LoongArch targets to the `dist-loongarch64-linux` job which has a ready-to-use LoongArch toolchain. With the proper CFLAGS applied it is possible to build artifacts suitable for bare-metal. I verified that the `compiler_builtins` objects are now correctly produced regarding architecture and ABI, with the changes here applied. Fixes rust-lang#125908. cc `@heiher` try-job: dist-loongarch64-linux try-job: dist-various-2
matthiaskrgr
added a commit
to matthiaskrgr/rust
that referenced
this issue
Jul 1, 2024
Fix x86_64 code being produced for bare-metal LoongArch targets' `compiler_builtins` Formerly the `loongarch*-*-none*` targets were added to the `dist-various-2` CI job, but no corresponding toolchain was added along with them. This meant the `compiler_builtins` for the targets were built with the host toolchain. As the other `dist-various` toolchains are mostly pre-built so far, to avoid burdening them with crosstool-ng builds, simply move the two bare-metal LoongArch targets to the `dist-loongarch64-linux` job which has a ready-to-use LoongArch toolchain. With the proper CFLAGS applied it is possible to build artifacts suitable for bare-metal. I verified that the `compiler_builtins` objects are now correctly produced regarding architecture and ABI, with the changes here applied. Fixes rust-lang#125908. cc `@heiher` try-job: dist-loongarch64-linux try-job: dist-various-2
matthiaskrgr
added a commit
to matthiaskrgr/rust
that referenced
this issue
Jul 1, 2024
Fix x86_64 code being produced for bare-metal LoongArch targets' `compiler_builtins` Formerly the `loongarch*-*-none*` targets were added to the `dist-various-2` CI job, but no corresponding toolchain was added along with them. This meant the `compiler_builtins` for the targets were built with the host toolchain. As the other `dist-various` toolchains are mostly pre-built so far, to avoid burdening them with crosstool-ng builds, simply move the two bare-metal LoongArch targets to the `dist-loongarch64-linux` job which has a ready-to-use LoongArch toolchain. With the proper CFLAGS applied it is possible to build artifacts suitable for bare-metal. I verified that the `compiler_builtins` objects are now correctly produced regarding architecture and ABI, with the changes here applied. Fixes rust-lang#125908. cc ``@heiher`` try-job: dist-loongarch64-linux try-job: dist-various-2
rust-timer
added a commit
to rust-lang-ci/rust
that referenced
this issue
Jul 1, 2024
Rollup merge of rust-lang#127150 - xen0n:issue125908, r=Kobzol Fix x86_64 code being produced for bare-metal LoongArch targets' `compiler_builtins` Formerly the `loongarch*-*-none*` targets were added to the `dist-various-2` CI job, but no corresponding toolchain was added along with them. This meant the `compiler_builtins` for the targets were built with the host toolchain. As the other `dist-various` toolchains are mostly pre-built so far, to avoid burdening them with crosstool-ng builds, simply move the two bare-metal LoongArch targets to the `dist-loongarch64-linux` job which has a ready-to-use LoongArch toolchain. With the proper CFLAGS applied it is possible to build artifacts suitable for bare-metal. I verified that the `compiler_builtins` objects are now correctly produced regarding architecture and ABI, with the changes here applied. Fixes rust-lang#125908. cc ``@heiher`` try-job: dist-loongarch64-linux try-job: dist-various-2
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
A-CI
Area: Our Github Actions CI
C-bug
Category: This is a bug.
O-loongarch
Target: LoongArch (LA32R, LA32S, LA64)
T-bootstrap
Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
When downloading the
loongarch64-unknown-none
(andsoftfloat
variant) target, thecompiler_builtins
that ships with the compiler is broken:Upon further inspection, the
compiler_builtins
symbols are indeed built for x86-64 and not LoongArch:It's not clear what causes this, but https://github.com/rust-lang/rust/blob/master/src/ci/docker/host-x86_64/dist-various-2/Dockerfile#L33 contains no corresponding definitions for
CC_
.This issue appears related, in that it would catch whatever causes this: #111142
The text was updated successfully, but these errors were encountered: