-
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
Create a generic AVR target: avr-unknown-unknown #131651
base: master
Are you sure you want to change the base?
Conversation
rustbot has assigned @compiler-errors. Use |
Some changes occurred in src/doc/rustc/src/platform-support cc @Noratrieb Some changes occurred in src/tools/compiletest cc @jieyouxu These commits modify compiler targets. |
cc @workingjubilee? |
This comment has been minimized.
This comment has been minimized.
bdabb36
to
2639188
Compare
This comment has been minimized.
This comment has been minimized.
2639188
to
dd8e00f
Compare
This comment has been minimized.
This comment has been minimized.
Ouch, I'm not sure why the test is failing - |
my first instinct is "bootstrap problem?" |
I'll take a look |
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.
The test and bootstrap side of things LGTM modulo the compiletest header test thing. Didn't look at the compiler side.
dd8e00f
to
1c8fe66
Compare
This comment has been minimized.
This comment has been minimized.
This commit removes the `avr-unknown-gnu-atmega328` target and replaces it with a more generic `avr-unknown-unknown` variant that can be specialized using `-C target-cpu` (e.g. `-C target-cpu=atmega328p`). I've decided to go with the `-unknown` tag (i.e. `avr-unknown-unknown` instead of `avr-unknown-gnu`), because that's the name LLVM already uses - I see no reason to diverge here. Seizing the day, I'm adding myself as the maintainer of this target - I've been already fixing the bugs anyway, might as well make it official. Related discussion: rust-lang#131171
1c8fe66
to
dcea80a
Compare
The job Click to see the possible cause of the failure (guessed by this bot)
|
This commit removes the
avr-unknown-gnu-atmega328
target and replaces it with a more genericavr-unknown-unknown
variant that can be specialized using-C target-cpu
(e.g.-C target-cpu=atmega328p
).I've decided to go with the
-unknown
tag (i.e.avr-unknown-unknown
instead ofavr-unknown-gnu
), because that's the name LLVM already uses - I see no reason to diverge here.Seizing the day, I'm adding myself as the maintainer of this target - I've been already fixing the bugs anyway, might as well make it official 🙂
Related discussion: #131171.