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 9 pull requests #128883

Merged
merged 30 commits into from
Aug 9, 2024
Merged

Rollup of 9 pull requests #128883

merged 30 commits into from
Aug 9, 2024

Commits on Aug 8, 2024

  1. Configuration menu
    Copy the full SHA
    fb1dac2 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    1e3ea29 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    2abcc6b View commit details
    Browse the repository at this point in the history
  4. Add Steal::is_stolen()

    Nadrieril committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    09ae438 View commit details
    Browse the repository at this point in the history
  5. Tweak wording

    Co-authored-by: lcnr <rust@lcnr.de>
    Nadrieril and lcnr authored Aug 8, 2024
    Configuration menu
    Copy the full SHA
    c966370 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    7c4150f View commit details
    Browse the repository at this point in the history

Commits on Aug 9, 2024

  1. Configuration menu
    Copy the full SHA
    11b801b View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a3f8edf View commit details
    Browse the repository at this point in the history
  3. fix format

    MinxuanZ committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    625432c View commit details
    Browse the repository at this point in the history
  4. delete space

    MinxuanZ committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    0106f5b View commit details
    Browse the repository at this point in the history
  5. VxWorks: Add safety comment for vxCpuEnabledGet

    Co-authored-by: Trevor Gross <t.gross35@gmail.com>
    biabbas and tgross35 authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    b20d9f0 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    b589f86 View commit details
    Browse the repository at this point in the history
  7. hir_typeck: use end_point over BytePos manipulations

    Parser has error recovery for Unicode-confusables, which includes the
    right parentheses `)`. If a multi-byte right parentheses look-alike
    reaches the argument removal suggestion diagnostics, it would trigger an
    assertion because the diagnostics used `- BytePos(1)` which can land
    within a multi-byte codepoint.
    
    This is fixed by using `SourceMap::end_point` to find the final right
    delimiter codepoint, which correctly respects codepoint boundaries.
    jieyouxu committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    879bfd7 View commit details
    Browse the repository at this point in the history
  8. tests: add regression test for rust-lang#128845

    For codepoint boundary assertion triggered by a let stmt compound
    assignment removal suggestion when encountering recovered multi-byte
    compound ops.
    
    Issue: <rust-lang#128845>
    jieyouxu committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    92520a9 View commit details
    Browse the repository at this point in the history
  9. parser: ensure let stmt compound assignment removal suggestion respec…

    …t codepoint boundaries
    
    Previously we would try to issue a suggestion for `let x <op>= 1`, i.e.
    a compound assignment within a `let` binding, to remove the `<op>`. The
    suggestion code unfortunately incorrectly assumed that the `<op>` is an
    exactly-1-byte ASCII character, but this assumption is incorrect because
    we also recover Unicode-confusables like `➖=` as `-=`. In this example,
    the suggestion code used a `+ BytePos(1)` to calculate the span of the
    `<op>` codepoint that looks like `-` but the mult-byte Unicode
    look-alike would cause the suggested removal span to be inside a
    multi-byte codepoint boundary, triggering a codepoint boundary
    assertion.
    
    Issue: <rust-lang#128845>
    jieyouxu committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    d65f131 View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    38874a6 View commit details
    Browse the repository at this point in the history
  11. add builder-config into tarball sources

    This will be useful for certain scenarios where developers want to know
    how the tarball sources were generated. We also want this to check for CI
    rustc incompatible options on bootstrap.
    
    Signed-off-by: onur-ozkan <work@onurozkan.dev>
    onur-ozkan committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    a7c415c View commit details
    Browse the repository at this point in the history
  12. use absolute path for config.toml

    Signed-off-by: onur-ozkan <work@onurozkan.dev>
    onur-ozkan committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    ff0d37c View commit details
    Browse the repository at this point in the history
  13. Configuration menu
    Copy the full SHA
    f72cb04 View commit details
    Browse the repository at this point in the history
  14. Configuration menu
    Copy the full SHA
    a380d5e View commit details
    Browse the repository at this point in the history
  15. Configuration menu
    Copy the full SHA
    1d19c2c View commit details
    Browse the repository at this point in the history
  16. Rollup merge of rust-lang#128815 - Nadrieril:is_stolen, r=jieyouxu,lcnr

    Add `Steal::is_stolen()`
    
    Writers of rustc drivers (such as myself) often encounter stealing issues. It is currently impossible to gracefully handle them. This PR adds a `Steal::is_stolen()` function for that purpose.
    matthiaskrgr authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    55329cc View commit details
    Browse the repository at this point in the history
  17. Rollup merge of rust-lang#128817 - biabbas:vxworks_update, r=tgross35

    VxWorks code refactored
    
    1. Extern TaskNameSet as minimum supported version of os is VxWorks 7 which would have taskNameSet
    2. Vx_TASK_NAME_LEN is 31 on VxWorks7, defined variable res.
    3. Add unsafe blocks on Non::Zero usage in available_parallelism()
    4. Update vxworks docs.
    r? `@tgross35`
    cc `@devnexen`
    matthiaskrgr authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    5372936 View commit details
    Browse the repository at this point in the history
  18. Rollup merge of rust-lang#128822 - onur-ozkan:add-build-config-in-tar…

    …balls, r=Kobzol
    
    add `builder-config` into tarball sources
    
    This will be useful for certain scenarios where developers want to know how the tarball sources were generated. We also want this to check for CI rustc incompatible options on bootstrap.
    
    Blocker for rust-lang#122709
    
    r? Kobzol
    matthiaskrgr authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    5f1e25e View commit details
    Browse the repository at this point in the history
  19. Rollup merge of rust-lang#128838 - notriddle:notriddle/invalid-tag-is…

    …-not-rust, r=GuillaumeGomez
    
    rustdoc: do not run doctests with invalid langstrings
    
    rust-lang#124577 (comment)
    
    CC ``@decathorpe``
    matthiaskrgr authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    9f11eb1 View commit details
    Browse the repository at this point in the history
  20. Rollup merge of rust-lang#128852 - folkertdev:multipart-suggestion-st…

    …able-sort, r=compiler-errors
    
    use stable sort to sort multipart diagnostics
    
    I think a stable sort should be used to sort the different parts of a multipart selection. The current unstable sort uses the text of the suggestion as a tie-breaker. That just doesn't seem right, and the order of the input is a better choice I think, because it gives the diagnostic author more control.
    
    This came up when I was building a suggestion where
    
    ```rust
    fn foo() {}
    ```
    
    must be turned into an unsafe function, and an attribute must be added
    
    ```rust
    #[target_feature(enable = "...")]
    unsafe fn foo() {}
    ```
    
    In this example, the two suggestions occur at the same position, but the order is extremely important: unsafe must come after the attribute. But the situation changes if there is a pub/pub(crate), and if the unsafe is already present. It just out that because of the suggestion text, there is no way for me to order the suggestions correctly.
    
    This change probably should be tested, but are there tests of the diagnostics code itself in the tests?
    
    r? ```@estebank```
    matthiaskrgr authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    2066002 View commit details
    Browse the repository at this point in the history
  21. Rollup merge of rust-lang#128859 - MinxuanZ:mips-sig, r=Amanieu

    Fix the name of signal 19 in library/std/src/sys/pal/unix/process/process_unix/tests.rs for mips/sparc linux
    
    relate to rust-lang#128816
    matthiaskrgr authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    2e0c5ad View commit details
    Browse the repository at this point in the history
  22. Rollup merge of rust-lang#128864 - jieyouxu:funnicode, r=Urgau

    Use `SourceMap::end_point` instead of `- BytePos(1)` in arg removal suggestion
    
    Previously, we tried to remove extra arg commas when providing extra arg removal suggestions. One of
    the edge cases is having to account for an arg that has a closing delimiter `)` following it.
    However, the previous suggestion code assumed that the delimiter is in fact exactly the 1-byte `)`
    character. This assumption was proven incorrect, because we recover from Unicode-confusable
    delimiters in the parser, which means that the ending delimiter could be a multi-byte codepoint
    that looks *like* a `)`. Subtracing 1 byte could land us in the middle of a codepoint, triggering a
    codepoint boundary assertion.
    
    This is fixed by using `SourceMap::end_point` which properly accounts for codepoint boundaries.
    
    Fixes rust-lang#128717.
    
    cc ````@fmease```` and rust-lang#128790
    matthiaskrgr authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    9eb77ac View commit details
    Browse the repository at this point in the history
  23. Rollup merge of rust-lang#128865 - jieyouxu:unicurd, r=Urgau

    Ensure let stmt compound assignment removal suggestion respect codepoint boundaries
    
    Previously we would try to issue a suggestion for `let x <op>= 1`, i.e.
    a compound assignment within a `let` binding, to remove the `<op>`. The
    suggestion code unfortunately incorrectly assumed that the `<op>` is an
    exactly-1-byte ASCII character, but this assumption is incorrect because
    we also recover Unicode-confusables like `➖=` as `-=`. In this example,
    the suggestion code used a `+ BytePos(1)` to calculate the span of the
    `<op>` codepoint that looks like `-` but the mult-byte Unicode
    look-alike would cause the suggested removal span to be inside a
    multi-byte codepoint boundary, triggering a codepoint boundary
    assertion.
    
    The fix is to use `SourceMap::start_point(token_span)` which properly accounts for codepoint boundaries.
    
    Fixes rust-lang#128845.
    
    cc rust-lang#128790
    
    r? ````@fmease````
    matthiaskrgr authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    665a1a4 View commit details
    Browse the repository at this point in the history
  24. Rollup merge of rust-lang#128874 - Kobzol:cmd-verbose-logging, r=onur…

    …-ozkan
    
    Disable verbose bootstrap command failure logging by default
    
    One of my recent bootstrap command refactoring PRs enabled verbose logging of command failures by default. While this is great for debugging bootstrap, in many situations it's just too verbose and prevents the user from seeing the actual printed stdout/stderr, which usually contains much more useful information.
    
    This PR reverts that logic, and only prints a detailed error when `-v` is passed to bootstrap.
    
    r? ````@onur-ozkan````
    matthiaskrgr authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    cea3b42 View commit details
    Browse the repository at this point in the history