-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Add new Tier-3 target: loongarch64-unknown-linux-ohos
#130750
Conversation
These commits modify compiler targets. Some changes occurred in src/doc/rustc/src/platform-support cc @Noratrieb |
This comment has been minimized.
This comment has been minimized.
1478a18
to
8a0c305
Compare
This comment has been minimized.
This comment has been minimized.
8a0c305
to
4f2704a
Compare
This comment has been minimized.
This comment has been minimized.
4f2704a
to
5d591c6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, r=me after the platform-support docs nit
5d591c6
to
9ab7046
Compare
MCP: rust-lang/compiler-team#784 Co-authored-by: WANG Rui <[email protected]>
@rustbot ready |
This comment was marked as outdated.
This comment was marked as outdated.
Actually wait, has the MCP finished? Or should this be |
The FCP 10-day waiting period is up. Could you label it as accepted? |
Oh, is it just that rustbot is not responding for whatever reason. |
@bors r+ rollup |
…ieyouxu Add new Tier-3 target: `loongarch64-unknown-linux-ohos` MCP: rust-lang/compiler-team#784
…mpiler-errors Rollup of 8 pull requests Successful merges: - rust-lang#129545 (rustdoc: redesign toolbar and disclosure widgets) - rust-lang#130618 (Skip query in get_parent_item when possible.) - rust-lang#130727 (Check vtable projections for validity in miri) - rust-lang#130739 (Pass bootstrap cargo when `--stage 0` and `COMPILETEST_FORCE_STAGE0`) - rust-lang#130750 (Add new Tier-3 target: `loongarch64-unknown-linux-ohos`) - rust-lang#130758 (Revert "Add recursion limit to FFI safety lint") - rust-lang#130759 (Update books) - rust-lang#130762 (stabilize const_intrinsic_copy) r? `@ghost` `@rustbot` modify labels: rollup
…mpiler-errors Rollup of 7 pull requests Successful merges: - rust-lang#129545 (rustdoc: redesign toolbar and disclosure widgets) - rust-lang#130618 (Skip query in get_parent_item when possible.) - rust-lang#130727 (Check vtable projections for validity in miri) - rust-lang#130750 (Add new Tier-3 target: `loongarch64-unknown-linux-ohos`) - rust-lang#130758 (Revert "Add recursion limit to FFI safety lint") - rust-lang#130759 (Update books) - rust-lang#130762 (stabilize const_intrinsic_copy) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#130750 - heiher:loong-linux-ohos-tier3, r=jieyouxu Add new Tier-3 target: `loongarch64-unknown-linux-ohos` MCP: rust-lang/compiler-team#784
|
||
pub(crate) fn target() -> Target { | ||
Target { | ||
// LLVM 15 doesn't support OpenHarmony yet, use a linux target instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: why does here currently still support for LLVM 15 when the minimum supported version is LLVM 18?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is copied from the aarch64-unknown-linux-ohos
target. The ohos
triple has been supported since LLVM 17, so it's time to update it. #130809
MCP: rust-lang/compiler-team#784