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

i386-gnu target for GNU/Hurd #172

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zamaudio
Copy link

This branch gets you most of the way to a complete port on GNU/Hurd.
There is a bug in rustc 1.29 sources that prevents this mrustc from compiling rustc on hurd.
(I believe the popfd and pushfd instructions are only valid when using .intel_syntax in GAS on i386)

We still need to port the libstd to os hurd target.

$ as
.syntax intel
^D
{standard input}: Assembler messages:
{standard input}:1: Error: unknown pseudo-op: `.syntax'

This commit fixes the above issue with the assembled instructions
when using the GNU assembler.
There is still a bug in rustc 1.29 that prevents this from building
rustc core.  But this commit gets you most of the way there.

We still need to port libstd to os hurd to complete the port.
@thepowersgang
Copy link
Owner

Do either of 1.39 or 1.54 support hurd?
If they do, then I'm OK with merging support.

Otherwise, I'm not willing (or really able) to (formally) support a target that rustc doesn't.

@bjorn3
Copy link
Contributor

bjorn3 commented Feb 14, 2022

Hurd isn't supported on any rustc version afaik, not even master has it: https://github.com/rust-lang/rust/tree/master/compiler/rustc_target/src/spec You may want to propose hurd as tier 3 target for rustc. See https://rustc-dev-guide.rust-lang.org/building/new-target.html for how to add it and https://doc.rust-lang.org/nightly/rustc/target-tier-policy.html#tier-3-target-policy for the tier 3 target policy you would have to follow.

@averyterrel
Copy link

https://doc.rust-lang.org/stable/rustc/platform-support/hurd.html
Hurd is currently supported in stable

@thepowersgang
Copy link
Owner

If that is part of 1.74, then that will be available once that branch is complete (see you in several months I guess)

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.

None yet

4 participants