Fixes wrong unreachable_pub lints on nested and glob public reexport #87487

lambinoo · 2021-07-26T17:40:54Z

Linked issues: #64762 & #82064

rust-highfive · 2021-07-26T17:40:57Z

(rust-highfive has picked a reviewer for you, use r? to override)

lambinoo · 2021-07-26T17:41:20Z

lambinoo · 2021-07-26T17:42:50Z

I've had one test weirdly failing locally in the debuginfo-gdb pass but I'm not too sure how it can be related to my change

petrochenkov · 2021-07-27T22:05:00Z

What I wanted to achieve with #82064 is assigning all Public and Exported levels in resolve, to all items, not only exports.
The EmbargoVisitor::access_levels table will be pre-initialized from the resolver table.
(After that we'll be able to add an assert to EmbargoVisitor::update making sure that only Reachable and ReachableFromImplTrait levels are inserted at that point.)

lambinoo · 2021-07-31T20:46:13Z

@petrochenkov Hey, good evening! I've been able to look at the necessary work during the week. Would that basically involve moving a big chunk of the code from the EmbargoVisitor in rustc_privacy to a new PrivacyVisitor in rustc_resolve?

I already have some local changes towards that goal but I prefer to ask before sinking too much time in the wrong direction

petrochenkov · 2021-07-31T21:07:47Z

Most of EmbargoVisitor is about Reachable and ReachableFromImplTrait so there's not much to move if you want to support Public and Exported only.
Only modules (in resolve sense, including enums and traits) need to be treated specially, all other items don't need to be visited in detail at all.

lambinoo · 2021-08-02T08:26:13Z

@petrochenkov Mmmh, I've noticed what I think is a major blocker.

As far as I know, I cannot correctly resolve the access level of Impl contained inside a Block from the Resolver. I need access to a TyCtxt for that.

rust/compiler/rustc_privacy/src/lib.rs

Lines 639 to 641 in 8759f00

    
           hir::ItemKind::Impl { .. } => { 
        
               Option::<AccessLevel>::of_impl(item.hir_id(), self.tcx, &self.access_levels) 
        
           }

https://github.com/rust-lang/rust/blob/86bd8913c953a34c8bd4fd59dbdd46620c5419a5/compiler/rustc_privacy/src/lib.rs#L357-L364

Current workarounds I found are:

being overly optimistic (and mark every pub items as Public) and rely on the types being private to throw errors
keeping a Public/Exported access level pass in rustc_privacy explicitely for Impl which is what I did
https://github.com/rust-lang/rust/blob/86bd8913c953a34c8bd4fd59dbdd46620c5419a5/compiler/rustc_privacy/src/lib.rs#L589-L598

My last push contains those changes (I still need to see how to handle macros so it's not finished yet)

petrochenkov · 2021-08-02T09:54:09Z

Ah, right, impls can also be public or exported.
You are right, you cannot correctly process impls early in resolve, the logic for impls will need to stay in rustc_privacy.

rustc: Replace `HirId`s with `LocalDefId`s in `AccessLevels` tables and passes using those tables - primarily privacy checking, stability checking and dead code checking. All these passes work with definitions rather than with arbitrary HIR nodes. r? `@cjgillot` cc `@lambinoo` (rust-lang#87487)

Mak DefId to AccessLevel map in resolve for export hir_id to accesslevel in resolve and applied in privacy using local def id removing tracing probes making function not recursive and adding comments Move most of Exported/Public res to rustc_resolve moving public/export res to resolve fix missing stability attributes in core, std and alloc move code to access_levels.rs return for some kinds instead of going through them Export correctness, macro changes, comments add comment for import binding add comment for import binding renmae to access level visitor, remove comments, move fn as closure, remove new_key fmt fix rebase fix rebase fmt fmt fix: move macro def to rustc_resolve fix: reachable AccessLevel for enum variants fmt fix: missing stability attributes for other architectures allow unreachable pub in rustfmt fix: missing impl access level + renaming export to reexport Missing impl access level was found thanks to a test in clippy

petrochenkov · 2022-01-10T00:34:46Z

@lambinoo
Has something changed?
I see that the clippy errors on CI are gone, but I don't see any changes in clippy.

lambinoo · 2022-01-10T05:32:34Z

I noticed a small mistake in the rustc_pricacy code. A line disappeared when I shuffled some code around.

I forgot to update the access level of the impl in visit_item (L650).

I found this in the most overkill way by having the compiler dump the access level structure and doing a diff between this branch and master... And a lot of impl that should have been public were missing.

petrochenkov · 2022-01-10T06:39:01Z

Great, hopefully it will pass now.
@bors r+

bors · 2022-01-10T06:39:02Z

📌 Commit 3a77bb8 has been approved by petrochenkov

bors · 2022-01-10T08:55:55Z

⌛ Testing commit 3a77bb8 with merge df035a3...

bors · 2022-01-10T11:53:09Z

☀️ Test successful - checks-actions
Approved by: petrochenkov
Pushing df035a3 to master...

rust-timer · 2022-01-10T13:24:29Z

Finished benchmarking commit (df035a3): comparison url.

Summary: This change led to moderate relevant improvements 🎉 in compiler performance.

Moderate improvement in instruction counts (up to -1.1% on incr-unchanged builds of unused-warnings)

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

@rustbot label: -perf-regression

lambinoo · 2022-01-10T13:27:24Z

Success! Gosh this one has been cooking for a while. I'll be working on improving the whole resolution thing now but I think the two linked issues can be closed now!
I wanted to thank you @petrochenkov (again x3) for the time you gave me for this PR and for your patience!

Pkgsrc changes: * Bump available bootstraps to 1.59.0. * Delete one patch which no longer applies, adjust another. Upstream changes: Version 1.60.0 (2022-04-07) ========================== Language -------- - [Stabilize `#[cfg(panic = "...")]` for either `"unwind"` or `"abort"`.] [93658] - [Stabilize `#[cfg(target_has_atomic = "...")]` for each integer size and `"ptr"`.][93824] Compiler -------- - [Enable combining `+crt-static` and `relocation-model=pic` on `x86_64-unknown-linux-gnu`][86374] - [Fixes wrong `unreachable_pub` lints on nested and glob public reexport][87487] - [Stabilize `-Z instrument-coverage` as `-C instrument-coverage`][90132] - [Stabilize `-Z print-link-args` as `--print link-args`][91606] - [Add new Tier 3 target `mips64-openwrt-linux-musl`\*][92300] - [Add new Tier 3 target `armv7-unknown-linux-uclibceabi` (softfloat)\*][92383] - [Fix invalid removal of newlines from doc comments][92357] - [Add kernel target for RustyHermit][92670] - [Deny mixing bin crate type with lib crate types][92933] - [Make rustc use `RUST_BACKTRACE=full` by default][93566] - [Upgrade to LLVM 14][93577] \* Refer to Rust's [platform support page][platform-support-doc] for more information on Rust's tiered platform support. Libraries --------- - [Guarantee call order for `sort_by_cached_key`][89621] - [Improve `Duration::try_from_secs_f32`/`f64` accuracy by directly processing exponent and mantissa][90247] - [Make `Instant::{duration_since, elapsed, sub}` saturating][89926] - [Remove non-monotonic clocks workarounds in `Instant::now`][89926] - [Make `BuildHasherDefault`, `iter::Empty` and `future::Pending` covariant][92630] Stabilized APIs --------------- - [`Arc::new_cyclic`][arc_new_cyclic] - [`Rc::new_cyclic`][rc_new_cyclic] - [`slice::EscapeAscii`][slice_escape_ascii] - [`<[u8]>::escape_ascii`][slice_u8_escape_ascii] - [`u8::escape_ascii`][u8_escape_ascii] - [`Vec::spare_capacity_mut`][vec_spare_capacity_mut] - [`MaybeUninit::assume_init_drop`][assume_init_drop] - [`MaybeUninit::assume_init_read`][assume_init_read] - [`i8::abs_diff`][i8_abs_diff] - [`i16::abs_diff`][i16_abs_diff] - [`i32::abs_diff`][i32_abs_diff] - [`i64::abs_diff`][i64_abs_diff] - [`i128::abs_diff`][i128_abs_diff] - [`isize::abs_diff`][isize_abs_diff] - [`u8::abs_diff`][u8_abs_diff] - [`u16::abs_diff`][u16_abs_diff] - [`u32::abs_diff`][u32_abs_diff] - [`u64::abs_diff`][u64_abs_diff] - [`u128::abs_diff`][u128_abs_diff] - [`usize::abs_diff`][usize_abs_diff] - [`Display for io::ErrorKind`][display_error_kind] - [`From<u8> for ExitCode`][from_u8_exit_code] - [`Not for !` (the "never" type)][not_never] - [_Op_`Assign<$t> for Wrapping<$t>`][wrapping_assign_ops] - [`arch::is_aarch64_feature_detected!`][is_aarch64_feature_detected] Cargo ----- - [Port cargo from `toml-rs` to `toml_edit`][cargo/10086] - [Stabilize `-Ztimings` as `--timings`][cargo/10245] - [Stabilize namespaced and weak dependency features.][cargo/10269] - [Accept more `cargo:rustc-link-arg-*` types from build script output.][cargo/10274] - [cargo-new should not add ignore rule on Cargo.lock inside subdirs][cargo/10379] Misc ---- - [Ship docs on Tier 2 platforms by reusing the closest Tier 1 platform docs][92800] - [Drop rustc-docs from complete profile][93742] - [bootstrap: tidy up flag handling for llvm build][93918] Compatibility Notes ------------------- - [Remove compiler-rt linking hack on Android][83822] - [Mitigations for platforms with non-monotonic clocks have been removed from `Instant::now`][89926]. On platforms that don't provide monotonic clocks, an instant is not guaranteed to be greater than an earlier instant anymore. - [`Instant::{duration_since, elapsed, sub}` do not panic anymore on underflow, saturating to `0` instead][89926]. In the real world the panic happened mostly on platforms with buggy monotonic clock implementations rather than catching programming errors like reversing the start and end times. Such programming errors will now results in `0` rather than a panic. - In a future release we're planning to increase the baseline requirements for the Linux kernel to version 3.2, and for glibc to version 2.17. We'd love your feedback in [PR #95026][95026]. Internal Changes ---------------- These changes provide no direct user facing benefits, but represent significant improvements to the internals and overall performance of rustc and related tools. - [Switch all libraries to the 2021 edition][92068] [83822]: rust-lang/rust#83822 [86374]: rust-lang/rust#86374 [87487]: rust-lang/rust#87487 [89621]: rust-lang/rust#89621 [89926]: rust-lang/rust#89926 [90132]: rust-lang/rust#90132 [90247]: rust-lang/rust#90247 [91606]: rust-lang/rust#91606 [92068]: rust-lang/rust#92068 [92300]: rust-lang/rust#92300 [92357]: rust-lang/rust#92357 [92383]: rust-lang/rust#92383 [92630]: rust-lang/rust#92630 [92670]: rust-lang/rust#92670 [92800]: rust-lang/rust#92800 [92933]: rust-lang/rust#92933 [93566]: rust-lang/rust#93566 [93577]: rust-lang/rust#93577 [93658]: rust-lang/rust#93658 [93742]: rust-lang/rust#93742 [93824]: rust-lang/rust#93824 [93918]: rust-lang/rust#93918 [95026]: rust-lang/rust#95026 [cargo/10086]: rust-lang/cargo#10086 [cargo/10245]: rust-lang/cargo#10245 [cargo/10269]: rust-lang/cargo#10269 [cargo/10274]: rust-lang/cargo#10274 [cargo/10379]: rust-lang/cargo#10379 [arc_new_cyclic]: https://doc.rust-lang.org/stable/std/sync/struct.Arc.html#method.new_cyclic [rc_new_cyclic]: https://doc.rust-lang.org/stable/std/rc/struct.Rc.html#method.new_cyclic [slice_escape_ascii]: https://doc.rust-lang.org/stable/std/slice/struct.EscapeAscii.html [slice_u8_escape_ascii]: https://doc.rust-lang.org/stable/std/primitive.slice.html#method.escape_ascii [u8_escape_ascii]: https://doc.rust-lang.org/stable/std/primitive.u8.html#method.escape_ascii [vec_spare_capacity_mut]: https://doc.rust-lang.org/stable/std/vec/struct.Vec.html#method.spare_capacity_mut [assume_init_drop]: https://doc.rust-lang.org/stable/std/mem/union.MaybeUninit.html#method.assume_init_drop [assume_init_read]: https://doc.rust-lang.org/stable/std/mem/union.MaybeUninit.html#method.assume_init_read [i8_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.i8.html#method.abs_diff [i16_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.i16.html#method.abs_diff [i32_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.i32.html#method.abs_diff [i64_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.i64.html#method.abs_diff [i128_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.i128.html#method.abs_diff [isize_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.isize.html#method.abs_diff [u8_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.u8.html#method.abs_diff [u16_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.u16.html#method.abs_diff [u32_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.u32.html#method.abs_diff [u64_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.u64.html#method.abs_diff [u128_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.u128.html#method.abs_diff [usize_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.usize.html#method.abs_diff [display_error_kind]: https://doc.rust-lang.org/stable/std/io/enum.ErrorKind.html#impl-Display [from_u8_exit_code]: https://doc.rust-lang.org/stable/std/process/struct.ExitCode.html#impl-From%3Cu8%3E [not_never]: https://doc.rust-lang.org/stable/std/primitive.never.html#impl-Not [wrapping_assign_ops]: https://doc.rust-lang.org/stable/std/num/struct.Wrapping.html#trait-implementations [is_aarch64_feature_detected]: https://doc.rust-lang.org/stable/std/arch/macro.is_aarch64_feature_detected.html

Pkgsrc changes: * Bump available bootstraps to 1.59.0. * Adjust line number in patches which had non-zero offsets. Upstream changes: Version 1.60.0 (2022-04-07) =========================== Language -------- - [Stabilize `#[cfg(panic = "...")]` for either `"unwind"` or `"abort"`.] [93658] - [Stabilize `#[cfg(target_has_atomic = "...")]` for each integer size and `"ptr"`.][93824] Compiler -------- - [Enable combining `+crt-static` and `relocation-model=pic` on `x86_64-unknown-linux-gnu`][86374] - [Fixes wrong `unreachable_pub` lints on nested and glob public reexport][87487] - [Stabilize `-Z instrument-coverage` as `-C instrument-coverage`][90132] - [Stabilize `-Z print-link-args` as `--print link-args`][91606] - [Add new Tier 3 target `mips64-openwrt-linux-musl`\*][92300] - [Add new Tier 3 target `armv7-unknown-linux-uclibceabi` (softfloat)\*][92383] - [Fix invalid removal of newlines from doc comments][92357] - [Add kernel target for RustyHermit][92670] - [Deny mixing bin crate type with lib crate types][92933] - [Make rustc use `RUST_BACKTRACE=full` by default][93566] - [Upgrade to LLVM 14][93577] \* Refer to Rust's [platform support page][platform-support-doc] for more information on Rust's tiered platform support. Libraries --------- - [Guarantee call order for `sort_by_cached_key`][89621] - [Improve `Duration::try_from_secs_f32`/`f64` accuracy by directly processing exponent and mantissa][90247] - [Make `Instant::{duration_since, elapsed, sub}` saturating][89926] - [Remove non-monotonic clocks workarounds in `Instant::now`][89926] - [Make `BuildHasherDefault`, `iter::Empty` and `future::Pending` covariant][92630] Stabilized APIs --------------- - [`Arc::new_cyclic`][arc_new_cyclic] - [`Rc::new_cyclic`][rc_new_cyclic] - [`slice::EscapeAscii`][slice_escape_ascii] - [`<[u8]>::escape_ascii`][slice_u8_escape_ascii] - [`u8::escape_ascii`][u8_escape_ascii] - [`Vec::spare_capacity_mut`][vec_spare_capacity_mut] - [`MaybeUninit::assume_init_drop`][assume_init_drop] - [`MaybeUninit::assume_init_read`][assume_init_read] - [`i8::abs_diff`][i8_abs_diff] - [`i16::abs_diff`][i16_abs_diff] - [`i32::abs_diff`][i32_abs_diff] - [`i64::abs_diff`][i64_abs_diff] - [`i128::abs_diff`][i128_abs_diff] - [`isize::abs_diff`][isize_abs_diff] - [`u8::abs_diff`][u8_abs_diff] - [`u16::abs_diff`][u16_abs_diff] - [`u32::abs_diff`][u32_abs_diff] - [`u64::abs_diff`][u64_abs_diff] - [`u128::abs_diff`][u128_abs_diff] - [`usize::abs_diff`][usize_abs_diff] - [`Display for io::ErrorKind`][display_error_kind] - [`From<u8> for ExitCode`][from_u8_exit_code] - [`Not for !` (the "never" type)][not_never] - [_Op_`Assign<$t> for Wrapping<$t>`][wrapping_assign_ops] - [`arch::is_aarch64_feature_detected!`][is_aarch64_feature_detected] Cargo ----- - [Port cargo from `toml-rs` to `toml_edit`][cargo/10086] - [Stabilize `-Ztimings` as `--timings`][cargo/10245] - [Stabilize namespaced and weak dependency features.][cargo/10269] - [Accept more `cargo:rustc-link-arg-*` types from build script output.][cargo/10274] - [cargo-new should not add ignore rule on Cargo.lock inside subdirs][cargo/10379] Misc ---- - [Ship docs on Tier 2 platforms by reusing the closest Tier 1 platform docs][92800] - [Drop rustc-docs from complete profile][93742] - [bootstrap: tidy up flag handling for llvm build][93918] Compatibility Notes ------------------- - [Remove compiler-rt linking hack on Android][83822] - [Mitigations for platforms with non-monotonic clocks have been removed from `Instant::now`][89926]. On platforms that don't provide monotonic clocks, an instant is not guaranteed to be greater than an earlier instant anymore. - [`Instant::{duration_since, elapsed, sub}` do not panic anymore on underflow, saturating to `0` instead][89926]. In the real world the panic happened mostly on platforms with buggy monotonic clock implementations rather than catching programming errors like reversing the start and end times. Such programming errors will now results in `0` rather than a panic. - In a future release we're planning to increase the baseline requirements for the Linux kernel to version 3.2, and for glibc to version 2.17. We'd love your feedback in [PR #95026][95026]. Internal Changes ---------------- These changes provide no direct user facing benefits, but represent significant improvements to the internals and overall performance of rustc and related tools. - [Switch all libraries to the 2021 edition][92068] [83822]: rust-lang/rust#83822 [86374]: rust-lang/rust#86374 [87487]: rust-lang/rust#87487 [89621]: rust-lang/rust#89621 [89926]: rust-lang/rust#89926 [90132]: rust-lang/rust#90132 [90247]: rust-lang/rust#90247 [91606]: rust-lang/rust#91606 [92068]: rust-lang/rust#92068 [92300]: rust-lang/rust#92300 [92357]: rust-lang/rust#92357 [92383]: rust-lang/rust#92383 [92630]: rust-lang/rust#92630 [92670]: rust-lang/rust#92670 [92800]: rust-lang/rust#92800 [92933]: rust-lang/rust#92933 [93566]: rust-lang/rust#93566 [93577]: rust-lang/rust#93577 [93658]: rust-lang/rust#93658 [93742]: rust-lang/rust#93742 [93824]: rust-lang/rust#93824 [93918]: rust-lang/rust#93918 [95026]: rust-lang/rust#95026 [cargo/10086]: rust-lang/cargo#10086 [cargo/10245]: rust-lang/cargo#10245 [cargo/10269]: rust-lang/cargo#10269 [cargo/10274]: rust-lang/cargo#10274 [cargo/10379]: rust-lang/cargo#10379 [arc_new_cyclic]: https://doc.rust-lang.org/stable/std/sync/struct.Arc.html#method.new_cyclic [rc_new_cyclic]: https://doc.rust-lang.org/stable/std/rc/struct.Rc.html#method.new_cyclic [slice_escape_ascii]: https://doc.rust-lang.org/stable/std/slice/struct.EscapeAscii.html [slice_u8_escape_ascii]: https://doc.rust-lang.org/stable/std/primitive.slice.html#method.escape_ascii [u8_escape_ascii]: https://doc.rust-lang.org/stable/std/primitive.u8.html#method.escape_ascii [vec_spare_capacity_mut]: https://doc.rust-lang.org/stable/std/vec/struct.Vec.html#method.spare_capacity_mut [assume_init_drop]: https://doc.rust-lang.org/stable/std/mem/union.MaybeUninit.html#method.assume_init_drop [assume_init_read]: https://doc.rust-lang.org/stable/std/mem/union.MaybeUninit.html#method.assume_init_read [i8_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.i8.html#method.abs_diff [i16_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.i16.html#method.abs_diff [i32_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.i32.html#method.abs_diff [i64_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.i64.html#method.abs_diff [i128_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.i128.html#method.abs_diff [isize_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.isize.html#method.abs_diff [u8_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.u8.html#method.abs_diff [u16_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.u16.html#method.abs_diff [u32_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.u32.html#method.abs_diff [u64_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.u64.html#method.abs_diff [u128_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.u128.html#method.abs_diff [usize_abs_diff]: https://doc.rust-lang.org/stable/std/primitive.usize.html#method.abs_diff [display_error_kind]: https://doc.rust-lang.org/stable/std/io/enum.ErrorKind.html#impl-Display [from_u8_exit_code]: https://doc.rust-lang.org/stable/std/process/struct.ExitCode.html#impl-From%3Cu8%3E [not_never]: https://doc.rust-lang.org/stable/std/primitive.never.html#impl-Not [wrapping_assign_ops]: https://doc.rust-lang.org/stable/std/num/struct.Wrapping.html#trait-implementations [is_aarch64_feature_detected]: https://doc.rust-lang.org/stable/std/arch/macro.is_aarch64_feature_detected.html

…rochenkov optimization of access level table construction Refactoring which was mentioned in rust-lang#87487

rust-highfive assigned nagisa Jul 26, 2021

rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 26, 2021

rust-highfive assigned petrochenkov and unassigned nagisa Jul 26, 2021

This comment has been minimized.

Sign in to view

lambinoo changed the title ~~Unreachable pub on nested uses~~ Fixes false positive unreachable_pub lints on nested and glob public reexport Jul 26, 2021

lambinoo changed the title ~~Fixes false positive unreachable_pub lints on nested and glob public reexport~~ Fixes wrong unreachable_pub lints on nested and glob public reexport Jul 26, 2021

lambinoo marked this pull request as ready for review July 27, 2021 13:57

lambinoo force-pushed the I-64762_unreachable_pub_lint branch from 32d0081 to 3c5d0e8 Compare July 27, 2021 14:17

petrochenkov mentioned this pull request Jul 28, 2021

rustc: Replace HirIds with LocalDefIds in AccessLevels tables #87568

Merged

petrochenkov added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 29, 2021

lambinoo force-pushed the I-64762_unreachable_pub_lint branch from 3c5d0e8 to 32eaff5 Compare August 2, 2021 08:26

lambinoo force-pushed the I-64762_unreachable_pub_lint branch from 32eaff5 to 86bd891 Compare August 2, 2021 08:27

This comment has been minimized.

Sign in to view

lambinoo force-pushed the I-64762_unreachable_pub_lint branch from 86bd891 to b888dc9 Compare August 4, 2021 15:49

This comment has been minimized.

Sign in to view

lambinoo force-pushed the I-64762_unreachable_pub_lint branch from aca14f2 to 3a77bb8 Compare January 9, 2022 21:33

bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jan 10, 2022

bors added the merged-by-bors This PR was explicitly merged by bors. label Jan 10, 2022

bors merged commit df035a3 into rust-lang:master Jan 10, 2022

rustbot added this to the 1.60.0 milestone Jan 10, 2022

rustbot removed the perf-regression Performance regression. label Jan 10, 2022

This was referenced Jan 10, 2022

Move collection of items accessible directly or through reexports from rustc_privacy to rustc_resolve #82064

Open

Lint unreachable_pub got easily confused by multiple items in a single pub use ... statement #64762

Closed

ehuss mentioned this pull request Jan 11, 2022

no MIR available for DefId #92755

Closed

petrochenkov mentioned this pull request Jan 12, 2022

Set struct/union/enum fields/variants as reachable when item is #92781

Merged

inquisitivecrystal added the relnotes Marks issues that should be documented in the release notes of the next release. label Jan 12, 2022

Bryanskiy mentioned this pull request Aug 4, 2022

optimization of access level table construction #100147

Merged

TaKO8Ki added a commit to TaKO8Ki/rust that referenced this pull request Aug 16, 2022

Rollup merge of rust-lang#100147 - Bryanskiy:private-in-public, r=pet…

7dfbaa2

…rochenkov optimization of access level table construction Refactoring which was mentioned in rust-lang#87487

TaKO8Ki added a commit to TaKO8Ki/rust that referenced this pull request Aug 16, 2022

Rollup merge of rust-lang#100147 - Bryanskiy:private-in-public, r=pet…

7f57787

…rochenkov optimization of access level table construction Refactoring which was mentioned in rust-lang#87487

Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Sep 2, 2022

Rollup merge of rust-lang#100147 - Bryanskiy:private-in-public, r=pet…

b39d4c6

…rochenkov optimization of access level table construction Refactoring which was mentioned in rust-lang#87487

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Sep 2, 2022

Rollup merge of rust-lang#100147 - Bryanskiy:private-in-public, r=pet…

1aaf9ae

…rochenkov optimization of access level table construction Refactoring which was mentioned in rust-lang#87487

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fixes wrong unreachable_pub lints on nested and glob public reexport #87487

Fixes wrong unreachable_pub lints on nested and glob public reexport #87487

lambinoo commented Jul 26, 2021

rust-highfive commented Jul 26, 2021

lambinoo commented Jul 26, 2021

lambinoo commented Jul 26, 2021

This comment has been minimized.

petrochenkov commented Jul 27, 2021

lambinoo commented Jul 31, 2021

petrochenkov commented Jul 31, 2021

lambinoo commented Aug 2, 2021 •

edited

Loading

This comment has been minimized.

petrochenkov commented Aug 2, 2021

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

petrochenkov commented Jan 10, 2022

lambinoo commented Jan 10, 2022 •

edited

Loading

petrochenkov commented Jan 10, 2022

bors commented Jan 10, 2022

bors commented Jan 10, 2022

bors commented Jan 10, 2022

rust-timer commented Jan 10, 2022

lambinoo commented Jan 10, 2022 •

edited

Loading

Fixes wrong unreachable_pub lints on nested and glob public reexport #87487

Fixes wrong unreachable_pub lints on nested and glob public reexport #87487

Conversation

lambinoo commented Jul 26, 2021

rust-highfive commented Jul 26, 2021

lambinoo commented Jul 26, 2021

lambinoo commented Jul 26, 2021

This comment has been minimized.

petrochenkov commented Jul 27, 2021

lambinoo commented Jul 31, 2021

petrochenkov commented Jul 31, 2021

lambinoo commented Aug 2, 2021 • edited Loading

This comment has been minimized.

petrochenkov commented Aug 2, 2021

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

petrochenkov commented Jan 10, 2022

lambinoo commented Jan 10, 2022 • edited Loading

petrochenkov commented Jan 10, 2022

bors commented Jan 10, 2022

bors commented Jan 10, 2022

bors commented Jan 10, 2022

rust-timer commented Jan 10, 2022

lambinoo commented Jan 10, 2022 • edited Loading

lambinoo commented Aug 2, 2021 •

edited

Loading

lambinoo commented Jan 10, 2022 •

edited

Loading

lambinoo commented Jan 10, 2022 •

edited

Loading