Auto merge of #100172 - Mark-Simulacrum:bump-165, r=Mark-Simulacrum

Bump to 1.65.0

r? `@Mark-Simulacrum`

Auto merge of #99743 - compiler-errors:fulfillment-context-cleanups, r=jackh726

Some `FulfillmentContext`-related cleanups

Use `ObligationCtxt` in some places, remove some `FulfillmentContext`s in others...

r? types

Auto merge of #100035 - workingjubilee:merge-functions, r=nikic

Enable function merging when opt is for size

It is, of course, natural to want to merge aliasing functions when
optimizing for code size, since that can eliminate several bytes.
And an exhaustive match helps make the code less brittle.

Closes #98215.

Enable function merging when opt is for size

It is, of course, natural to want to merge aliasing functions when
optimizing for code size, since that can eliminate several bytes.
And an exhaustive match helps make the code less brittle.

Auto merge of #100174 - Dylan-DPC:rollup-wnskbk6, r=Dylan-DPC

Rollup of 6 pull requests

Successful merges:

 - #99835 (Suggest adding/removing `ref` for binding patterns)
 - #100155 (Use `node_type_opt` to skip over generics that were not expected)
 - #100157 (rustdoc: use `collect()` instead of repeatedly pushing)
 - #100158 (kmc-solid: Add a stub implementation of #98246 (`File::set_times`))
 - #100166 (Remove more Clean trait implementations)
 - #100168 (Improve diagnostics for `const a: = expr;`)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup

Rollup merge of #100168 - WaffleLapkin:improve_diagnostics_for_missing_type_in_a_const_item, r=compiler-errors

Improve diagnostics for `const a: = expr;`

Adds a suggestion to write a type when there is a colon, but the type is not present.
I've also shrunk spans a little, so the suggestions are a little nicer.

Resolves #100146

r? `@compiler-errors`

Rollup merge of #100166 - GuillaumeGomez:rm-clean-impls, r=Dylan-DPC

Remove more Clean trait implementations

Follow-up of https://github.com/rust-lang/rust/pull/99638.

r? `@notriddle`

Rollup merge of #100158 - solid-rs:patch/kmc-solid/follow-up-98246, r=joshtriplett

kmc-solid: Add a stub implementation of #98246 (`File::set_times`)

 Fixes the build failure of the [`*-kmc-solid_*`](https://doc.rust-lang.org/nightly/rustc/platform-support/kmc-solid.html) Tier 3 targets after #98246.

This target does not support setting a modification time and access time separately, hence stubbing out the implementation.

Rollup merge of #100157 - rust-lang:notriddle/use-map-instead-of-repeated-push, r=Dylan-DPC

rustdoc: use `collect()` instead of repeatedly pushing

Rollup merge of #100155 - compiler-errors:issue-100154, r=jackh726

Use `node_type_opt` to skip over generics that were not expected

Fixes #100154

Rollup merge of #99835 - TaKO8Ki:suggest-adding-or-removing-ref-for-binding-pattern, r=estebank

Suggest adding/removing `ref` for binding patterns

This fixes what a fixme comment says.

r? `@estebank`

Bump to 1.65.0

Auto merge of #100073 - dpaoliello:externvar, r=michaelwoerister

Add test for raw-dylib with an external variable

All existing tests of link kind `raw-dylib` only validate the ability to link against functions, but it is also possible to link against variables.

This adds tests for linking against a variable using `raw-dylib` both by-name and by-ordinal.

Improve diagnostics for `const a: = expr;`

remove Clean trait implementation for hir::TraitItem

remove Clean trait implementation for hir::PolyTraitRef

Auto merge of #95977 - FabianWolff:issue-92790-dead-tuple, r=estebank

Warn about dead tuple struct fields

Continuation of #92972. Fixes #92790.

The language team has already commented on this in https://github.com/rust-lang/rust/pull/92972#issuecomment-1021511970; I have incorporated their requests here. Specifically, there is now a new allow-by-default `unused_tuple_struct_fields` lint (name bikesheddable), and fields of unit type are ignored (https://github.com/rust-lang/rust/pull/92972#issuecomment-1021815408), so error messages look like this:
```
error: field is never read: `1`
  --> $DIR/tuple-struct-field.rs:6:21
   |
LL | struct Wrapper(i32, [u8; LEN], String);
   |                     ^^^^^^^^^
   |
help: change the field to unit type to suppress this warning while preserving the field numbering
   |
LL | struct Wrapper(i32, (), String);
   |                     ~~
```
r? `@joshtriplett`

Auto merge of #99867 - spastorino:refactor-remap-lifetimes, r=nikomatsakis

Split create_def and lowering of lifetimes for opaque types and bare async fns

r? `@cjgillot`

This work is kind of half-way, but I think it could be merged anyway.
I think we should be able to remove all the vacant arms in `new_named_lifetime_with_res`, if I'm not wrong that requires visiting more nodes. We can do that as a follow up.
In follow-up PRs, besides the thing mentioned previously, I'll be trying to remove `LifetimeCaptureContext`, `captured_lifetimes` as a global data structure, global `binders_to_ignore` and all their friends :).

Also try to remap in a more general way based on def-ids.

Auto merge of #97085 - rylev:test-issue-33172, r=wesleywiser

Add a test for issue #33172

Adds a test confirming that #33172 has been fixed.

CDB has some surprising results as it looks like the supposedly unmangled static's symbol name is prefixed when it shouldn't be.

r? `@wesleywiser`

Closes #33172

rustdoc: use `collect()` instead of repeatedly pushing to bounds

rustdoc: use `collect()` instead of repeatedly pushing to bindings

Auto merge of #95026 - cuviper:bump-linux-min, r=Mark-Simulacrum

Increase the minimum linux-gnu versions

This is implementing the MCP from rust-lang/compiler-team#493. It is
increasing the minimum requirements of a couple Tier 1 targets, and
others at lower tiers, so this should go through FCP sign-offs for both
`T-compiler` and `T-release`.

The new `linux-gnu` baseline is kernel 3.2 and glibc 2.17. We will also
take that kernel as the minimum floor for _all_ `*-linux-*` targets, so
it may be broadly assumed in the implementation of the standard library.
That does not preclude specific targets from having greater requirements
where it makes sense, like a new arch needing something newer, or a
platform like `linux-android` choosing a newer baseline.

Link /rustroot/lib stuff for clang -m32 to see

opt node type

Auto merge of #100151 - matthiaskrgr:rollup-irqwvj2, r=matthiaskrgr

Rollup of 7 pull requests

Successful merges:

 - #98796 (Do not exclusively suggest `;` when `,` is also a choice)
 - #99772 (Re-enable submodule archive downloads.)
 - #100058 (Suggest a positional formatting argument instead of a captured argument)
 - #100093 (Enable unused_parens for match arms)
 - #100095 (More EarlyBinder cleanups)
 - #100138 (Remove more Clean trait implementations)
 - #100148 (RustWrapper: update for TypedPointerType in LLVM)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup

Rollup merge of #100148 - durin42:llvm-16-pointertype, r=nikic

RustWrapper: update for TypedPointerType in LLVM

This is a result of https://reviews.llvm.org/D130592.

r? `@nikic`

Rollup merge of #100138 - GuillaumeGomez:rm-clean-impls, r=notriddle

Remove more Clean trait implementations

Follow-up of https://github.com/rust-lang/rust/pull/99638.

r? `@notriddle`

Rollup merge of #100095 - jackh726:early-binder, r=lcnr

More EarlyBinder cleanups

Each commit is independent

r? types

Rollup merge of #100093 - wcampbell0x2a:unused-parens-for-match-arms, r=petrochenkov

Enable unused_parens for match arms

Fixes: https://github.com/rust-lang/rust/issues/92751
Currently I can't get the `stderr` to work with `./x.py test`, but this should fix the issue. Help would be appreciated!

Rollup merge of #100058 - TaKO8Ki:suggest-positional-formatting-argument-instead-of-format-args-capture, r=estebank

Suggest a positional formatting argument instead of a captured argument

This patch fixes a part of #96999.

fixes #98241
fixes #97311

r? `@estebank`

Rollup merge of #99772 - ehuss:reenable-submodule-archive, r=Mark-Simulacrum

Re-enable submodule archive downloads.

This is effectively a revert of #98423 (though it keeps the `--depth 1` flag since that is still helpful).

GitHub has indicated that they have been working on the original issue, and my testing shows that the llvm-project archive download now succeeds 100% of the time.

This should save about a minute on every job.

Rollup merge of #98796 - compiler-errors:no-semi-if-comma, r=estebank

Do not exclusively suggest `;` when `,` is also a choice

Fixes #96791

Add test for raw-dylib with an external variable

Fix typo

Improve opt_local_def_id docs

Downgrade dist-powerpc-linux binutils to 2.30

With binutils 2.32, we were getting errors like this:

    relocation truncated to fit: R_PPC_PLTREL24 against symbol
        `__cxa_atexit@@GLIBC_2.1.3' defined in .plt section in
        /x-tools/powerpc-unknown-linux-gnu/powerpc-unknown-linux-gnu/sysroot/usr/lib/crt1.o

but it builds okay with binutils 2.30.

Improve record_def_id_remap docs

Do not collect lifetimes with Infer resolution

RustWrapper: update for TypedPointerType in LLVM

This is a result of https://reviews.llvm.org/D130592.

Use span_bug instead of panic

Add docs to generics_def_id_map

Add docs to record_elided_anchor

Extract record_elided_anchor

Move new_remapping inside with_hir_id_owner

Add documentation for create_lifetime_defs

Add more debug calls

Add documentation about lifetime args

Add documentation about lifetime_defs

Move hir_bounds after lifetime_defs

captures -> collected_lifetimes

Move lifetimes_in_bounds call to outside with_hir_id_owner block in lower_async_fn_ret_ty

Move lifetimes_in_bounds call to outside with_hir_id_owner block in lower_opaque_impl_trait

Document what collected_lifetimes vec containts

Document lower_opaque_impl_trait

with_lifetime_binder is now lower_lifetime_binder and doesn't need a closure

Add comments on with_remapping

Document opt_local_def_id

Document generics_def_id_map field and record/get methods on it

Add comments about lifetime collect and create lifetime defs for RPITs

Avoid explicitly handling res when is not needed

create_and_capture_lifetime_defs -> create_lifetime_defs

Remove captured_lifetimes and LifetimeCaptureContext and make create_lifetime_defs return the captures

Implement def_id based remapping

Remove local_def_id from captured_lifetimes

Capture things as Lifetime object to simplify things

Create new_mapping local structure and avoid checking def_ids on captures

Remove binders_to_ignore from LifetimeCaptureContext

No need to store parent_def_id in LifetimeCaptureContext

No need to check binders to ignore on new_named_lifetime_with_res's LifetimeRes::Fresh

No need to check binders to ignore on new_named_lifetime_with_res's LifetimeRes::Param

new_named_lifetime_with_res's LifetimeRes::Fresh should have created def_id already

new_named_lifetime_with_res's LifetimeRes::Param should have created def_id already

Fail if LifetimeRes is Infer when creating def ids for lifetimes copies

Record RPITs elided lifetimes in Rptr Tys

Record RPITs elided lifetimes in path segments

Restructure visit_ty in a more clear way

Skip lifetimes in binders when visiting

Move LifetimeCollectVisitor to rustc_ast_lowering

Completely remove captures flag

Do not execute captures code when lowering lifetimes as GenericArg

Do not execute captures code for async fns

There's no need to check binders_to_ignore using the old code anymore

Explicitly gather lifetimes and definitions for bare async fns

Extract create_and_capture_lifetime_defs function

Remove NEW_COLLECT_LIFETIMES env var

Add debug! calls

Explicitly gather lifetimes and definitions in RPIT

Unroll while_capturing_lifetimes into lower_opaque_impl_trait

Add captures flag to capture or not while lowering

Make lower_generic_bound_predicate receive AST bounds instead of HIR bounds

Remove old docs on lower_async_fn_ret_ty

Address nits

Add `traits::fully_solve_obligation` that acts like `traits::fully_normalize`

It spawns up a trait engine, registers the single obligation, then fully
solves it

Use ObligationCtxt in impossible_predicates

Remove unnecessary FulfillmentContext from need_migrate_deref_output_trait_object

Remove FulfillmentContext param from fully_normalize

Use ObligationCtxt in main fn return type check

Auto merge of #100087 - JakobDegen:mir-patch, r=tmiasko

Avoid invalidating the CFG in `MirPatch`

As a part of this change, we adjust `MirPatch` to not needlessly create unnecessary resume blocks.

r? `@tmiasko`

add a comment about what we can parse now

Enable unused_parens for match arms