rust/compiler/rustc_resolve/src, branch 1.84.0 https://github.com/rust-lang/rust http://git.dreamy.place/mirrors/rust/atom?h=1.84.0 2024-11-20T18:57:02+00:00 Store resolution for self and crate root module segments 2024-11-20T18:57:02+00:00 Michael Goulet michael@errs.io 2024-10-27T00:16:44+00:00 urn:sha1:19b528b8a0770d55ba3c7b3cc5b91d6466e6ff2d Auto merge of #132761 - nnethercote:resolve-tweaks, r=petrochenkov 2024-11-19T19:54:35+00:00 bors bors@rust-lang.org 2024-11-19T19:54:35+00:00 urn:sha1:ee612c45f00391aff71ec0c52b7fc08fae18c711 Resolve tweaks A couple of small perf improvements, and some minor refactorings, all in `rustc_resolve`. r? `@petrochenkov` Rollup merge of #132787 - maxcabrajac:fnctxt, r=petrochenkov 2024-11-16T20:05:46+00:00 Matthias Krüger matthias.krueger@famsik.de 2024-11-16T20:05:46+00:00 urn:sha1:6b47c6d786df3fd5141174ee1fe04ec2fd957e3d Unify FnKind between AST visitors and make WalkItemKind more straight forward Unifying `FnKind` requires a bunch of changes to `WalkItemKind::walk` signature so I'll change them in one go related to #128974 r? `@petrochenkov` Make WalkItemKind::walk signature compatible between Visitor versions 2024-11-15T20:01:53+00:00 maxcabrajac max@cabrajac.com 2024-11-12T16:20:17+00:00 urn:sha1:516a3b0c9b7f2a77a40db7d85479d4ef568bd261 Add WalkItemKind::Ctxt so AssocCtxt is not sent to non-Assoc ItemKinds 2024-11-15T20:00:01+00:00 maxcabrajac max@cabrajac.com 2024-11-08T21:51:28+00:00 urn:sha1:6180173612e7a8de35db441cde14c3cfacc62af7 Remove `Resolver::empty_disambiguator`. 2024-11-14T01:08:20+00:00 Nicholas Nethercote n.nethercote@gmail.com 2024-11-14T00:34:49+00:00 urn:sha1:12747f188a9dba7b13cf1ccb01fef74c6ce3294e It was added in #115367 for anonymous ADTs. Those changes were then reverted in #131045, but `empty_disambiguator` was left behind, perhaps by mistake. It seems to be unnecessary. Replace the `restricted_shadowing` boolean argument with an enum. 2024-11-14T00:35:27+00:00 Nicholas Nethercote n.nethercote@gmail.com 2024-11-13T23:20:45+00:00 urn:sha1:66cc7d6a05f2fadbe100e12ebc65d511fa690aea It makes the code clearer. Move a call outside a loop. 2024-11-14T00:35:27+00:00 Nicholas Nethercote n.nethercote@gmail.com 2024-11-08T04:24:13+00:00 urn:sha1:6e0e9edbe034b8ce764f91abb5c12b7871e09e9c This path isn't hot enough for this to affect performance, but there's no point repeating the same computation multiple times. Remove two `_ext` methods. 2024-11-14T00:35:27+00:00 Nicholas Nethercote n.nethercote@gmail.com 2024-11-08T04:16:16+00:00 urn:sha1:d71c06d022b07518f97072ad36cbe2c8a25eee2d `resolve_ident_in_module` is a very thin wrapper around `resolve_ident_in_module_ext`, and `resolve_ident_in_module_unadjusted` is a very thin wrapper around `resolve_ident_in_module_unadjusted_ext`. The wrappers make the call sites slightly more concise, but I don't think that's worth the extra code and indirection. This commit removes the two wrappers and removes the `_ext` suffixes from the inner methods. Use an atom comparison for a keyword check. 2024-11-14T00:35:27+00:00 Nicholas Nethercote n.nethercote@gmail.com 2024-11-07T23:42:07+00:00 urn:sha1:e7dffeedcffce81f709da8a076b8ff437b8d3060 Instead of a string comparison.