-
Notifications
You must be signed in to change notification settings - Fork 303
Announcing Rust 1.88.0 #1651
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
Announcing Rust 1.88.0 #1651
Conversation
Hi relnotes-interest-group, this PR adds a release blog post. Could you review cc @alex-semenyuk @jieyouxu @joshtriplett @Kobzol @lcnr @traviscross |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
Co-authored-by: 许杰友 Jieyou Xu (Joe) <[email protected]> Co-authored-by: Jubilee <[email protected]> Co-authored-by: Trevor Gross <[email protected]>
Expand const-stabilized API links in relnotes Noticed while looking at the relnotes blog post rust-lang/blog.rust-lang.org#1651 (comment). r? `@cuviper`
Expand const-stabilized API links in relnotes Noticed while looking at the relnotes blog post rust-lang/blog.rust-lang.org#1651 (comment). r? ``@cuviper``
Expand const-stabilized API links in relnotes Noticed while looking at the relnotes blog post rust-lang/blog.rust-lang.org#1651 (comment). r? ```@cuviper```
Co-authored-by: 许杰友 Jieyou Xu (Joe) <[email protected]>
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.
Looks good to me
Deployment is done! 🚀 |
|
||
```rust | ||
#[unsafe(naked)] | ||
pub unsafe extern "sysv64" fn wrapping_add(a: u64, b: u64) { |
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.
Should this have a return type?
pub unsafe extern "sysv64" fn wrapping_add(a: u64, b: u64) -> u64 {
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.
it should (technically it's okay as it is, but it is confusing and unintentional)
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.
Please feel free to send a PR correction!
pub unsafe extern "sysv64" fn wrapping_add(a: u64, b: u64) { | ||
// Equivalent to `a.wrapping_add(b)`. | ||
core::arch::naked_asm!( | ||
"add rax, rdi, rsi", |
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 does not seem valid in x86-64 which only has 2-operand add. Or does it mean something like:
"add rax, rdi, rsi", | |
"lea rax, [rdi + rsi]", |
(not checked)
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.
ah, I've been led astray by godbolt, which will assembly the 3-operand add, but fail to execute it. I tried to not use lea
just because add
is more clear, but now to use the 2-operand add
you'd have to mov. Also there is just a rdi + rsi
in there so it's probably sufficiently clear.
fix at #1654
Expand const-stabilized API links in relnotes Noticed while looking at the relnotes blog post rust-lang/blog.rust-lang.org#1651 (comment). r? ````@cuviper````
Rollup merge of #143033 - jieyouxu:expand-apis, r=cuviper Expand const-stabilized API links in relnotes Noticed while looking at the relnotes blog post rust-lang/blog.rust-lang.org#1651 (comment). r? ````@cuviper````
cc @rust-lang/release
@rustbot ping relnotes-interest-group
Rendered