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

rustc: Set MIPS cpu/features in the compiler #31303

Merged
merged 1 commit into from
Feb 1, 2016

Conversation

alexcrichton
Copy link
Member

Currently any compilation to MIPS spits out the warning:

'generic' is not a recognized processor for this target (ignoring processor)

Doesn't make for a great user experience! We don't encounter this in the normal
bootstrap because the cpu/feature set are set by the makefiles. Instead let's
just propagate these to the defaults for the entire target all the time (still
overridable from the command line) and prevent warnings from being emitted by
default.

Currently any compilation to MIPS spits out the warning:

    'generic' is not a recognized processor for this target (ignoring processor)

Doesn't make for a great user experience! We don't encounter this in the normal
bootstrap because the cpu/feature set are set by the makefiles. Instead let's
just propagate these to the defaults for the entire target all the time (still
overridable from the command line) and prevent warnings from being emitted by
default.
@rust-highfive
Copy link
Collaborator

r? @jroesch

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member Author

Also to be clear, I don't know what these cpus/features are, I'm just propagating what was there originally.

r? @aturon

@rust-highfive rust-highfive assigned aturon and unassigned jroesch Jan 30, 2016
japaric pushed a commit to japaric/rust that referenced this pull request Jan 30, 2016
@aturon
Copy link
Member

aturon commented Feb 1, 2016

@bors: r+

@bors
Copy link
Contributor

bors commented Feb 1, 2016

📌 Commit 0316013 has been approved by aturon

@bors
Copy link
Contributor

bors commented Feb 1, 2016

⌛ Testing commit 0316013 with merge 91e8044...

bors added a commit that referenced this pull request Feb 1, 2016
Currently any compilation to MIPS spits out the warning:

    'generic' is not a recognized processor for this target (ignoring processor)

Doesn't make for a great user experience! We don't encounter this in the normal
bootstrap because the cpu/feature set are set by the makefiles. Instead let's
just propagate these to the defaults for the entire target all the time (still
overridable from the command line) and prevent warnings from being emitted by
default.
@bors bors merged commit 0316013 into rust-lang:master Feb 1, 2016
@alexcrichton alexcrichton deleted the mips-warnings branch February 8, 2016 23:59
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.

5 participants