-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Fixup Apple target's description strings #130650
Fixup Apple target's description strings #130650
Conversation
These commits modify compiler targets. |
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 comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
@bors rollup |
compiler/rustc_target/src/spec/targets/aarch64_apple_ios_macabi.rs
Outdated
Show resolved
Hide resolved
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.
Don't have much opinion on this, looks good other than the Mac Catalyst thing noted
d40f412
to
114093c
Compare
Thanks! |
…kingjubilee Rollup of 5 pull requests Successful merges: - rust-lang#130648 (move enzyme flags from general cargo to rustc-specific cargo) - rust-lang#130650 (Fixup Apple target's description strings) - rust-lang#130664 (Generate line numbers for non-rust code examples as well) - rust-lang#130665 (Prevent Deduplication of `LongRunningWarn`) - rust-lang#130669 (tests: Test that `extern "C" fn` ptrs lint on slices) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#130650 - BlackHoleFox:apple-target-desc-consistency, r=jieyouxu Fixup Apple target's description strings Noticed this inconsistency in how the Apple target's had their new descriptions written while looking at rust-lang#130614, and figured it was easy enough to fixup shortly. I think prefixing every OS with `Apple` is clearer, especially for less known ones like `visionOS` and `watchOS`; so that's what was done here along with making the architecture names more consistent and then some other small tweaks. ~~r? `@thomcc~~` cc `@madsmtm`
Noticed this inconsistency in how the Apple target's had their new descriptions written while looking at #130614, and figured it was easy enough to fixup shortly. I think prefixing every OS with
Apple
is clearer, especially for less known ones likevisionOS
andwatchOS
; so that's what was done here along with making the architecture names more consistent and then some other small tweaks.r? @thomcccc @madsmtm