Skip to content
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

Rollup of 8 pull requests #108920

Merged
merged 19 commits into from
Mar 9, 2023
Merged

Rollup of 8 pull requests #108920

merged 19 commits into from
Mar 9, 2023

Commits on Mar 4, 2023

  1. Configuration menu
    Copy the full SHA
    118afdf View commit details
    Browse the repository at this point in the history

Commits on Mar 5, 2023

  1. Configuration menu
    Copy the full SHA
    bfefd11 View commit details
    Browse the repository at this point in the history

Commits on Mar 7, 2023

  1. Remove DropAndReplace terminator

    PR 107844 made DropAndReplace unused, let's remove it completely
    from the codebase.
    zeegomo committed Mar 7, 2023
    Configuration menu
    Copy the full SHA
    c5d4e4d View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    8d13454 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    3bfcfd0 View commit details
    Browse the repository at this point in the history
  4. remove leftover comment

    zeegomo committed Mar 7, 2023
    Configuration menu
    Copy the full SHA
    153bfa0 View commit details
    Browse the repository at this point in the history

Commits on Mar 8, 2023

  1. Configuration menu
    Copy the full SHA
    08e5a77 View commit details
    Browse the repository at this point in the history
  2. Tweak E0740

    compiler-errors committed Mar 8, 2023
    Configuration menu
    Copy the full SHA
    64eea3c View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    ead8b96 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    a74e651 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    7e932db View commit details
    Browse the repository at this point in the history
  6. Rollup merge of rust-lang#108754 - compiler-errors:retry, r=oli-obk

    Retry `pred_known_to_hold_modulo_regions` with fulfillment if ambiguous
    
    Fixes rust-lang#108721
    
    The problem here is that when we're checking `is_sized_raw` during codegen on some type that has a lot of opaques in it, something emits several nested obligations that are individually ambiguous, but when processed together in a loop then apply modulo regions. Since the `evaluate_predicates_recursively` inner loop doesn't process predicates until they stop changing, we return `EvaluatedToAmbig`, which makes the sized check return false incorrectly. See:
    
    https://github.com/rust-lang/rust/blob/f15f0ea73972786e426732c5b92ba9a904b866c4/compiler/rustc_trait_selection/src/traits/select/mod.rs#L596-L606
    
    ... Compared to the analogous loop in the new solver:
    
    https://github.com/rust-lang/rust/blob/f15f0ea73972786e426732c5b92ba9a904b866c4/compiler/rustc_trait_selection/src/solve/mod.rs#L481-L512
    
    To fix this, if we get ambiguous during `pred_known_to_hold_modulo_regions`, just retry the obligation in a fulfillment context.
    
    --
    
    Unfortunately... I don't have a test for this. I've only tested this locally. Pending minimization :/
    
    r? types
    matthiaskrgr authored Mar 8, 2023
    Configuration menu
    Copy the full SHA
    9408af9 View commit details
    Browse the repository at this point in the history
  7. Rollup merge of rust-lang#108759 - workingjubilee:pedantically-correc…

    …t-release-notes-for-1.41.1-apple32, r=Mark-Simulacrum
    
    1.41.1 supported 32-bit Apple targets
    
    This question was raised by rust-lang#108556.
    matthiaskrgr authored Mar 8, 2023
    Configuration menu
    Copy the full SHA
    23f46c5 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#108839 - compiler-errors:canonicalize-the-r…

    …oot-var, r=lcnr
    
    Canonicalize root var when making response from new solver
    
    During trait solving, if we equate two inference variables `?0` and `?1` but don't equate them with any rigid types, then `InferCtxt::probe_ty_var` will return `Err` for both of these. The canonicalizer code will then canonicalize the variables independently(!), and the response will not reflect the fact that these two variables have been made equal.
    
    This hinders inference and I also don't think it's sound? I haven't thought too much about it past that, so let's talk about it.
    
    r? ``@lcnr``
    matthiaskrgr authored Mar 8, 2023
    Configuration menu
    Copy the full SHA
    2428083 View commit details
    Browse the repository at this point in the history
  9. Rollup merge of rust-lang#108856 - Zeegomo:remove-drop-and-rep, r=tmi…

    …asko
    
    Remove DropAndReplace terminator
    
    rust-lang#107844 made DropAndReplace unused, let's remove it completely from the codebase.
    matthiaskrgr authored Mar 8, 2023
    Configuration menu
    Copy the full SHA
    4e84fbf View commit details
    Browse the repository at this point in the history
  10. Rollup merge of rust-lang#108882 - compiler-errors:E0740, r=eholk

    Tweak E0740
    
    Also drive-by suppress E0740 if it's an unresolved type.
    matthiaskrgr authored Mar 8, 2023
    Configuration menu
    Copy the full SHA
    e6e82db View commit details
    Browse the repository at this point in the history
  11. Rollup merge of rust-lang#108898 - ferrocene:pa-libc-check-cfg, r=Mar…

    …k-Simulacrum
    
    Set `LIBC_CHECK_CFG=1` when building Rust code in bootstrap
    
    Downstream forks of the Rust compiler might want to use a custom `libc` to add support for targets that are not yet available upstream. Adding a patch to replace `libc` with a custom one would cause compilation errors though, because Cargo would interpret the custom `libc` as part of the workspace, and apply the check-cfg lints on it.
    
    Since rust-lang/libc#3037, the `libc` build script emits check-cfg flags only when the `LIBC_CHECK_CFG` environment variable is set, so this PR allows the use of custom `libc`s.
    matthiaskrgr authored Mar 8, 2023
    Configuration menu
    Copy the full SHA
    2ebb5b1 View commit details
    Browse the repository at this point in the history
  12. Rollup merge of rust-lang#108911 - GuillaumeGomez:improve-rustdoc-gui…

    …-tester-code, r=notriddle
    
    Improve rustdoc-gui/tester.js code a bit
    
    Just a small clean-up.
    
    r? `@notriddle`
    matthiaskrgr authored Mar 8, 2023
    Configuration menu
    Copy the full SHA
    e6f39f7 View commit details
    Browse the repository at this point in the history
  13. Rollup merge of rust-lang#108916 - fmease:rm-unused-ret-val, r=compil…

    …er-errors
    
    Remove an unused return value in `rustc_hir_typeck`
    
    Unused since rust-lang#89580. Just something I noticed a while ago.
    
    `@rustbot` label C-cleanup
    matthiaskrgr authored Mar 8, 2023
    Configuration menu
    Copy the full SHA
    a22c5f9 View commit details
    Browse the repository at this point in the history