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 7 pull requests #122777

Closed
wants to merge 19 commits into from

Commits on Mar 19, 2024

  1. Report arm intersections

    Nadrieril committed Mar 19, 2024
    Configuration menu
    Copy the full SHA
    3dfd0fd View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    e4487ad View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    d697dd4 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    45cb9d8 View commit details
    Browse the repository at this point in the history
  5. Use the default file permissions when writing

    static libraries with ar_archive_writer
    
    Fixes rust-lang#107495
    bjorn3 committed Mar 19, 2024
    Configuration menu
    Copy the full SHA
    2a805f5 View commit details
    Browse the repository at this point in the history
  6. use more accurate terminology

    rustc is just one tool/executable, even if at the center of the toolchain
    tshepang authored Mar 19, 2024
    Configuration menu
    Copy the full SHA
    75a5196 View commit details
    Browse the repository at this point in the history

Commits on Mar 20, 2024

  1. fix OOB pointer formed in Vec::index

    Move the length check to before using `index` with `ptr::add` to prevent
    an out of bounds pointer from being formed.
    
    Fixes rust-lang#122760
    jwong101 committed Mar 20, 2024
    Configuration menu
    Copy the full SHA
    37718f9 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    70206f0 View commit details
    Browse the repository at this point in the history
  3. coverage: Remove incorrect assertions from counter allocation

    These assertions detect situations where a BCB node would have both a physical
    counter and one or more in-edge counters/expressions.
    
    For most BCBs that situation would indicate an implementation bug. However,
    it's perfectly fine in the case of a BCB having an edge that loops back to
    itself.
    
    Given the complexity and risk involved in fixing the assertions, and the fact
    that nothing relies on them actually being true, this patch just removes them
    instead.
    Zalathar committed Mar 20, 2024
    Configuration menu
    Copy the full SHA
    85bec7a View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    2f21e4f View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    92f668c View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    98e6655 View commit details
    Browse the repository at this point in the history
  7. Rollup merge of rust-lang#122644 - Nadrieril:complexity-tests, r=comp…

    …iler-errors
    
    pattern analysis: add a custom test harness
    
    There are two features of the pattern analysis code that are hard to test: the newly-added pattern complexity limit, and the computation of arm intersections. This PR adds some crate-specific tests for that, including an unmaintainable but pretty macro to help construct patterns.
    
    r? `@compiler-errors`
    GuillaumeGomez authored Mar 20, 2024
    Configuration menu
    Copy the full SHA
    01b16b5 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#122723 - bjorn3:archive_writer_fixes, r=nne…

    …thercote
    
    Use same file permissions for ar_archive_writer as the LLVM archive writer
    
    This is required to switch to ar_archive_writer in the future without regressions. In addition to this PR support for reading thin archives needs to be added (rust-lang#107407) to fix all known regressions.
    
    Fixes rust-lang#107495
    GuillaumeGomez authored Mar 20, 2024
    Configuration menu
    Copy the full SHA
    e779e97 View commit details
    Browse the repository at this point in the history
  9. Rollup merge of rust-lang#122740 - tshepang:patch-1, r=clubby789

    use more accurate terminology
    
    rustc is just one tool/executable, even if at the center of the toolchain
    GuillaumeGomez authored Mar 20, 2024
    Configuration menu
    Copy the full SHA
    ae90533 View commit details
    Browse the repository at this point in the history
  10. Rollup merge of rust-lang#122761 - jwong101:fix/vec-insert, r=working…

    …jubilee,Nilstrieb
    
    fix OOB pointer formed in Vec::index
    
    Move the length check to before using `index` with `ptr::add` to prevent an out of bounds pointer from being formed.
    
    Fixes rust-lang#122760
    GuillaumeGomez authored Mar 20, 2024
    Configuration menu
    Copy the full SHA
    c9fb47a View commit details
    Browse the repository at this point in the history
  11. Rollup merge of rust-lang#122764 - Zalathar:loopy, r=oli-obk

    coverage: Remove incorrect assertions from counter allocation
    
    These assertions detect situations where a BCB node (in the coverage graph) would have both a physical counter and one or more in-edge counters/expressions.
    
    For most BCBs that situation would indicate an implementation bug. However, it's perfectly fine in the case of a BCB having an edge that loops back to itself.
    
    Given the complexity and risk involved in fixing the assertions, and the fact that nothing relies on them actually being true, this patch just removes them instead.
    
    Fixes rust-lang#122738.
    
    `@rustbot` label +A-code-coverage
    GuillaumeGomez authored Mar 20, 2024
    Configuration menu
    Copy the full SHA
    6c92e04 View commit details
    Browse the repository at this point in the history
  12. Rollup merge of rust-lang#122765 - workingjubilee:test-for-vec-handli…

    …ng-usize-max, r=Nilstrieb
    
    Add `usize::MAX` arg tests for Vec
    
    Tests to prevent recurrence of the UB from the rust-lang#122760 issue.
    
    I skipped the `with_capacity`, `drain`, `reserve`, etc. APIs because they actually had a good assortment of tests earlier in the same file.
    
    r? Nilstrieb
    GuillaumeGomez authored Mar 20, 2024
    Configuration menu
    Copy the full SHA
    991f252 View commit details
    Browse the repository at this point in the history
  13. Configuration menu
    Copy the full SHA
    c55ca2e View commit details
    Browse the repository at this point in the history