Reduce precedence of expressions that have an outer attrPreviously, `-Zunpretty=expanded` would expand this program as follows:```rust#![feature(stmt_expr_attributes)]macro_rules! repro {    ($e:expr) => {        #[allow(deprecated)] $e    };}#[derive(Default)]struct Thing {    #[deprecated]    field: i32,}fn main() {    let thing = Thing::default();    let _ = repro!(thing).field;}``````rs#![feature(prelude_import)]#![feature(stmt_expr_attributes)]#[prelude_import]use std::prelude::rust_2021::*;#[macro_use]extern crate std;struct Thing {    #[deprecated]    field: i32,}#[automatically_derived]impl ::core::default::Default for Thing {    #[inline]    fn default() -> Thing {        Thing { field: ::core::default::Default::default() }    }}fn main() {    let thing = Thing::default();    let _ = #[allow(deprecated)] thing.field;}```This is not the correct expansion. The correct output would have `(#[allow(deprecated)] thing).field` with the attribute applying only to `thing`, not to `thing.field`.

Rollup of 10 pull requestsSuccessful merges: -#133952 (Remove wasm legacy abi) -#134661 (Reduce precedence of expressions that have an outer attr) -#141769 (Move metadata object generation for dylibs to the linker code ) -#141864 (Handle win32 separator for cygwin paths) -#142347 (Async drop - fix for StorageLive/StorageDead codegen for pinned future) -#142377 (Try unremapping compiler sources) -#142389 (Apply ABI attributes on return types in `rustc_codegen_cranelift`) -#142470 (Add some missing mailmap entries) -#142481 (Add `f16` inline asm support for LoongArch) -#142509 (bump std libc dependency)r? `@ghost``@rustbot` modify labels: rollup

Reduce precedence of expressions that have an outer attrPreviously, `-Zunpretty=expanded` would expand this program as follows:```rust#![feature(stmt_expr_attributes)]macro_rules! repro {    ($e:expr) => {        #[allow(deprecated)] $e    };}#[derive(Default)]struct Thing {    #[deprecated]    field: i32,}fn main() {    let thing = Thing::default();    let _ = repro!(thing).field;}``````rs#![feature(prelude_import)]#![feature(stmt_expr_attributes)]#[prelude_import]use std::prelude::rust_2021::*;#[macro_use]extern crate std;struct Thing {    #[deprecated]    field: i32,}#[automatically_derived]impl ::core::default::Default for Thing {    #[inline]    fn default() -> Thing {        Thing { field: ::core::default::Default::default() }    }}fn main() {    let thing = Thing::default();    let _ = #[allow(deprecated)] thing.field;}```This is not the correct expansion. The correct output would have `(#[allow(deprecated)] thing).field` with the attribute applying only to `thing`, not to `thing.field`.

Rollup of 9 pull requestsSuccessful merges: -#133952 (Remove wasm legacy abi) -#134661 (Reduce precedence of expressions that have an outer attr) -#141769 (Move metadata object generation for dylibs to the linker code ) -#141864 (Handle win32 separator for cygwin paths) -#141937 (Report never type lints in dependencies) -#142347 (Async drop - fix for StorageLive/StorageDead codegen for pinned future) -#142389 (Apply ABI attributes on return types in `rustc_codegen_cranelift`) -#142470 (Add some missing mailmap entries) -#142481 (Add `f16` inline asm support for LoongArch)r? `@ghost``@rustbot` modify labels: rollup

Rollup of 10 pull requestsSuccessful merges: -#133952 (Remove wasm legacy abi) -#134661 (Reduce precedence of expressions that have an outer attr) -#141769 (Move metadata object generation for dylibs to the linker code ) -#141937 (Report never type lints in dependencies) -#142347 (Async drop - fix for StorageLive/StorageDead codegen for pinned future) -#142389 (Apply ABI attributes on return types in `rustc_codegen_cranelift`) -#142470 (Add some missing mailmap entries) -#142481 (Add `f16` inline asm support for LoongArch) -#142499 (Remove check run bootstrap) -#142543 (Suggest adding semicolon in user code rather than macro impl details)r? `@ghost``@rustbot` modify labels: rollup

Rollup merge of#134661 - dtolnay:prefixattr, r=fmeaseReduce precedence of expressions that have an outer attrPreviously, `-Zunpretty=expanded` would expand this program as follows:```rust#![feature(stmt_expr_attributes)]macro_rules! repro {    ($e:expr) => {        #[allow(deprecated)] $e    };}#[derive(Default)]struct Thing {    #[deprecated]    field: i32,}fn main() {    let thing = Thing::default();    let _ = repro!(thing).field;}``````rs#![feature(prelude_import)]#![feature(stmt_expr_attributes)]#[prelude_import]use std::prelude::rust_2021::*;#[macro_use]extern crate std;struct Thing {    #[deprecated]    field: i32,}#[automatically_derived]impl ::core::default::Default for Thing {    #[inline]    fn default() -> Thing {        Thing { field: ::core::default::Default::default() }    }}fn main() {    let thing = Thing::default();    let _ = #[allow(deprecated)] thing.field;}```This is not the correct expansion. The correct output would have `(#[allow(deprecated)] thing).field` with the attribute applying only to `thing`, not to `thing.field`.

Insert parentheses around binary operation with attributeFixes the bug found by `@fmease` inrust-lang#134661 (review).Previously, `-Zunpretty=expanded` would expand this program as follows:```rust#![feature(stmt_expr_attributes)]#![allow(unused_attributes)]macro_rules! group {    ($e:expr) => {        $e    };}macro_rules! extra {    ($e:expr) => {        #[allow()] $e    };}fn main() {    let _ = #[allow()] 1 + 1;    let _ = group!(#[allow()] 1) + 1;    let _ = 1 + group!(#[allow()] 1);    let _ = extra!({ 0 }) + 1;    let _ = extra!({ 0 } + 1);}``````consolelet _ = #[allow()] 1 + 1;let _ = #[allow()] 1 + 1;let _ = 1 + #[allow()] 1;let _ = #[allow()] { 0 } + 1;let _ = #[allow()] { 0 } + 1;```The first 4 statements are the correct expansion, but the last one is not. The attribute is supposed to apply to the entire binary operation, not only to the left operand.After this PR, the 5th statement will expand to:```consolelet _ = #[allow()] ({ 0 } + 1);```In the future, as some subset of `stmt_expr_attributes` approaches stabilization, it is possible that we will need to do parenthesization for a number of additional cases depending on the outcome ofrust-lang#127436. But for now, at least this PR makes the pretty-printer align with the current behavior of the parser.r? fmease

Rollup merge of#142476 - dtolnay:attrbinop, r=fmeaseInsert parentheses around binary operation with attributeFixes the bug found by `@fmease` in#134661 (review).Previously, `-Zunpretty=expanded` would expand this program as follows:```rust#![feature(stmt_expr_attributes)]#![allow(unused_attributes)]macro_rules! group {    ($e:expr) => {        $e    };}macro_rules! extra {    ($e:expr) => {        #[allow()] $e    };}fn main() {    let _ = #[allow()] 1 + 1;    let _ = group!(#[allow()] 1) + 1;    let _ = 1 + group!(#[allow()] 1);    let _ = extra!({ 0 }) + 1;    let _ = extra!({ 0 } + 1);}``````consolelet _ = #[allow()] 1 + 1;let _ = #[allow()] 1 + 1;let _ = 1 + #[allow()] 1;let _ = #[allow()] { 0 } + 1;let _ = #[allow()] { 0 } + 1;```The first 4 statements are the correct expansion, but the last one is not. The attribute is supposed to apply to the entire binary operation, not only to the left operand.After this PR, the 5th statement will expand to:```consolelet _ = #[allow()] ({ 0 } + 1);```In the future, as some subset of `stmt_expr_attributes` approaches stabilization, it is possible that we will need to do parenthesization for a number of additional cases depending on the outcome of#127436. But for now, at least this PR makes the pretty-printer align with the current behavior of the parser.r? fmease

Reduce precedence of expressions that have an outer attrPreviously, `-Zunpretty=expanded` would expand this program as follows:```rust#![feature(stmt_expr_attributes)]macro_rules! repro {    ($e:expr) => {        #[allow(deprecated)] $e    };}#[derive(Default)]struct Thing {    #[deprecated]    field: i32,}fn main() {    let thing = Thing::default();    let _ = repro!(thing).field;}``````rs#![feature(prelude_import)]#![feature(stmt_expr_attributes)]#[prelude_import]use std::prelude::rust_2021::*;#[macro_use]extern crate std;struct Thing {    #[deprecated]    field: i32,}#[automatically_derived]impl ::core::default::Default for Thing {    #[inline]    fn default() -> Thing {        Thing { field: ::core::default::Default::default() }    }}fn main() {    let thing = Thing::default();    let _ = #[allow(deprecated)] thing.field;}```This is not the correct expansion. The correct output would have `(#[allow(deprecated)] thing).field` with the attribute applying only to `thing`, not to `thing.field`.

Rollup of 10 pull requestsSuccessful merges: -rust-lang/rust#133952 (Remove wasm legacy abi) -rust-lang/rust#134661 (Reduce precedence of expressions that have an outer attr) -rust-lang/rust#141769 (Move metadata object generation for dylibs to the linker code ) -rust-lang/rust#141937 (Report never type lints in dependencies) -rust-lang/rust#142347 (Async drop - fix for StorageLive/StorageDead codegen for pinned future) -rust-lang/rust#142389 (Apply ABI attributes on return types in `rustc_codegen_cranelift`) -rust-lang/rust#142470 (Add some missing mailmap entries) -rust-lang/rust#142481 (Add `f16` inline asm support for LoongArch) -rust-lang/rust#142499 (Remove check run bootstrap) -rust-lang/rust#142543 (Suggest adding semicolon in user code rather than macro impl details)r? `@ghost``@rustbot` modify labels: rollup

Rollup of 10 pull requestsSuccessful merges: -rust-lang/rust#133952 (Remove wasm legacy abi) -rust-lang/rust#134661 (Reduce precedence of expressions that have an outer attr) -rust-lang/rust#141769 (Move metadata object generation for dylibs to the linker code ) -rust-lang/rust#141937 (Report never type lints in dependencies) -rust-lang/rust#142347 (Async drop - fix for StorageLive/StorageDead codegen for pinned future) -rust-lang/rust#142389 (Apply ABI attributes on return types in `rustc_codegen_cranelift`) -rust-lang/rust#142470 (Add some missing mailmap entries) -rust-lang/rust#142481 (Add `f16` inline asm support for LoongArch) -rust-lang/rust#142499 (Remove check run bootstrap) -rust-lang/rust#142543 (Suggest adding semicolon in user code rather than macro impl details)r? `@ghost``@rustbot` modify labels: rollup