Skip to content

Commit

Permalink
hermitkernel-target: Set OS to "none"
Browse files Browse the repository at this point in the history
For our kernel targets, we should not set OS, as the kernel runs bare
metal without a circular dependency on std.

This also prepares us for unifying with
#89062. This patch requires
libhermit-rs to change a `cfg`s from `target_os = "hermit"` to `target_os
= "none"`.

I tested this patch locally.
  • Loading branch information
mkroening committed Oct 29, 2021
1 parent 37f70a0 commit 311a249
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion compiler/rustc_target/src/spec/hermit_kernel_base.rs
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ pub fn opts() -> TargetOptions {
);

TargetOptions {
os: "hermit".to_string(),
linker_flavor: LinkerFlavor::Lld(LldFlavor::Ld),
disable_redzone: true,
linker: Some("rust-lld".to_owned()),
Expand Down

0 comments on commit 311a249

Please sign in to comment.