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 #130265

Closed
wants to merge 17 commits into from
Closed

Commits on Sep 5, 2024

  1. Update compiler-builtins to 0.1.125

    This commit updates the compiler-builtins crate from 0.1.123 to 0.1.125.
    The changes in this update are:
    
    * rust-lang/compiler-builtins#682
    * rust-lang/compiler-builtins#678
    * rust-lang/compiler-builtins#685
    alexcrichton committed Sep 5, 2024
    Configuration menu
    Copy the full SHA
    5396124 View commit details
    Browse the repository at this point in the history

Commits on Sep 6, 2024

  1. Don't leave debug locations for constants sitting on the builder inde…

    …finitely.
    
    Because constants are currently emitted *before* the prologue, leaving the
    debug location on the IRBuilder spills onto other instructions in the prologue
    and messes up both line numbers as well as the point LLVM chooses to be the
    prologue end.
    
    Example LLVM IR (irrelevant IR elided):
    Before:
    
    define internal { i64, i64 } @_ZN3tmp3Foo18var_return_opt_try17he02116165b0fc08cE(ptr align 8 %self) !dbg !347 {
    start:
      %self.dbg.spill = alloca [8 x i8], align 8
      %_0 = alloca [16 x i8], align 8
      %residual.dbg.spill = alloca [0 x i8], align 1
        #dbg_declare(ptr %residual.dbg.spill, !353, !DIExpression(), !357)
      store ptr %self, ptr %self.dbg.spill, align 8, !dbg !357
        #dbg_declare(ptr %self.dbg.spill, !350, !DIExpression(), !358)
    
    After:
    
    define internal { i64, i64 } @_ZN3tmp3Foo18var_return_opt_try17h00b17d08874ddd90E(ptr align 8 %self) !dbg !347 {
    start:
      %self.dbg.spill = alloca [8 x i8], align 8
      %_0 = alloca [16 x i8], align 8
      %residual.dbg.spill = alloca [0 x i8], align 1
        #dbg_declare(ptr %residual.dbg.spill, !353, !DIExpression(), !357)
      store ptr %self, ptr %self.dbg.spill, align 8
        #dbg_declare(ptr %self.dbg.spill, !350, !DIExpression(), !358)
    
    Note in particular how !357 from %residual.dbg.spill's dbg_declare no longer
    falls through onto the store to %self.dbg.spill. This fixes argument values
    at entry when the constant is a ZST (e.g. <Option as Try>::Residual). This
    fixes rust-lang#130003 (but note that it does *not* fix issues with argument values and
    non-ZST constants, which emit their own stores that have debug info on them,
    like rust-lang#128945).
    khuey committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    7ed9f94 View commit details
    Browse the repository at this point in the history

Commits on Sep 9, 2024

  1. Fix default/minimum deployment target for Aarch64 simulator targets

    The minimum that `rustc` encoded did not match the version in Clang, and
    that meant that that when linking, we ended up bumping the version.
    
    Specifically, this sets the correct deployment target of the following
    simulator and Mac Catalyst targets:
    - `aarch64-apple-ios-sim` from 10.0 to 14.0
    - `aarch64-apple-tvos-sim` from 10.0 to 14.0
    - `aarch64-apple-watchos-sim` from 5.0 to 7.0
    - `aarch64-apple-ios-macabi` from 13.1 to 14.0
    
    I have chosen to not document the simulator target versions in the
    platform support docs, as it is fundamentally uninteresting; the normal
    targets (e.g. `aarch64-apple-ios`, `aarch64-apple-tvos`) still have the
    same deployment target as before, and that's what developers should
    actually target.
    madsmtm committed Sep 9, 2024
    Configuration menu
    Copy the full SHA
    97df8fb View commit details
    Browse the repository at this point in the history
  2. Fix slice::first_mut docs

    pointer -> reference
    Scripter17 committed Sep 9, 2024
    Configuration menu
    Copy the full SHA
    99cad12 View commit details
    Browse the repository at this point in the history

Commits on Sep 10, 2024

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

Commits on Sep 11, 2024

  1. Configuration menu
    Copy the full SHA
    6d06429 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    594de02 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    8dc2278 View commit details
    Browse the repository at this point in the history

Commits on Sep 12, 2024

  1. Configuration menu
    Copy the full SHA
    ed9b2ba View commit details
    Browse the repository at this point in the history
  2. Rollup merge of rust-lang#129367 - madsmtm:fix-apple-aarch64-deployme…

    …nt-targets, r=jieyouxu
    
    Fix default/minimum deployment target for Aarch64 simulator targets
    
    The minimum that `rustc` encoded did not match [the version in Clang](https://github.com/llvm/llvm-project/blob/llvmorg-18.1.8/llvm/lib/TargetParser/Triple.cpp#L1900-L1932), and that meant that that when linking, Clang ended up bumping the version. See rust-lang#129432 for more motivation behind this change.
    
    Specifically, this PR sets the correct deployment target of the following targets:
    - `aarch64-apple-ios-sim` from 10.0 to 14.0
    - `aarch64-apple-tvos-sim` from 10.0 to 14.0
    - `aarch64-apple-watchos-sim` from 5.0 to 7.0
    - `aarch64-apple-ios-macabi` from 13.1 to 14.0
    
    I have chosen not to document the `-sim` changes in the platform support docs, as it is fundamentally uninteresting; the normal targets (e.g. `aarch64-apple-ios`) still have the same deployment target, and that's what developers should actually target.
    
    r? compiler
    
    CC `@BlackHoleFox`
    Zalathar authored Sep 12, 2024
    Configuration menu
    Copy the full SHA
    956c18c View commit details
    Browse the repository at this point in the history
  3. Rollup merge of rust-lang#129992 - alexcrichton:update-compiler-built…

    …ins, r=tgross35
    
    Update compiler-builtins to 0.1.125
    
    This commit updates the compiler-builtins crate from 0.1.123 to 0.1.125. The changes in this update are:
    
    * rust-lang/compiler-builtins#682
    * rust-lang/compiler-builtins#678
    * rust-lang/compiler-builtins#685
    Zalathar authored Sep 12, 2024
    Configuration menu
    Copy the full SHA
    0e06c1d View commit details
    Browse the repository at this point in the history
  4. Rollup merge of rust-lang#130052 - khuey:clear-dilocation-after-const…

    …-emission, r=michaelwoerister
    
    Don't leave debug locations for constants sitting on the builder indefinitely
    
    Because constants are currently emitted *before* the prologue, leaving the debug location on the IRBuilder spills onto other instructions in the prologue and messes up both line numbers as well as the point LLVM chooses to be the prologue end.
    
    Example LLVM IR (irrelevant IR elided):
    Before:
    ```
    define internal { i64, i64 } ```@_ZN3tmp3Foo18var_return_opt_try17he02116165b0fc08cE(ptr``` align 8 %self) !dbg !347 { start:
      %self.dbg.spill = alloca [8 x i8], align 8
      %_0 = alloca [16 x i8], align 8
      %residual.dbg.spill = alloca [0 x i8], align 1
        #dbg_declare(ptr %residual.dbg.spill, !353, !DIExpression(), !357)
      store ptr %self, ptr %self.dbg.spill, align 8, !dbg !357
        #dbg_declare(ptr %self.dbg.spill, !350, !DIExpression(), !358)
    ```
    After:
    ```
    define internal { i64, i64 } ```@_ZN3tmp3Foo18var_return_opt_try17h00b17d08874ddd90E(ptr``` align 8 %self) !dbg !347 { start:
      %self.dbg.spill = alloca [8 x i8], align 8
      %_0 = alloca [16 x i8], align 8
      %residual.dbg.spill = alloca [0 x i8], align 1
        #dbg_declare(ptr %residual.dbg.spill, !353, !DIExpression(), !357)
      store ptr %self, ptr %self.dbg.spill, align 8
        #dbg_declare(ptr %self.dbg.spill, !350, !DIExpression(), !358)
    ```
    Note in particular how !357 from %residual.dbg.spill's dbg_declare no longer falls through onto the store to %self.dbg.spill. This fixes argument values at entry when the constant is a ZST (e.g. `<Option as Try>::Residual`). This fixes rust-lang#130003 (but note that it does *not* fix issues with argument values and non-ZST constants, which emit their own stores that have debug info on them, like rust-lang#128945).
    
    r? ```@michaelwoerister```
    Zalathar authored Sep 12, 2024
    Configuration menu
    Copy the full SHA
    aa83aac View commit details
    Browse the repository at this point in the history
  5. Rollup merge of rust-lang#130156 - nebulark:test_buildinfo, r=jieyouxu

    Add test for S_OBJNAME & update test for LF_BUILDINFO cl and cmd
    
    Update the unit test for checking cl and cmd in LF_BUILDINFO. With llvm-pdbutil we can now more specifically check if the string appears at the right location instead of just checking whether the string exists at all.
    Context: rust-lang#96475
    Zalathar authored Sep 12, 2024
    Configuration menu
    Copy the full SHA
    05f9ded View commit details
    Browse the repository at this point in the history
  6. Rollup merge of rust-lang#130160 - Scripter17:fix-slice-first_mut-doc…

    …, r=Amanieu
    
    Fix `slice::first_mut` docs
    
    Changes `pointer` to `reference` since that's the actual type it returns.
    
    `slice::last_mut` does correctly say "reference"
    Zalathar authored Sep 12, 2024
    Configuration menu
    Copy the full SHA
    d62fc5a View commit details
    Browse the repository at this point in the history
  7. Rollup merge of rust-lang#130250 - compiler-errors:useless-conversion…

    …, r=jieyouxu
    
    Fix `clippy::useless_conversion`
    
    Self-explanatory. Probably the last clippy change I'll actually put up since this is the only other one I've actually seen in the wild.
    Zalathar authored Sep 12, 2024
    Configuration menu
    Copy the full SHA
    d0f8865 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#130252 - compiler-errors:const-gen, r=cheny…

    …ukang
    
    Properly report error on `const gen fn`
    
    Fixes rust-lang#130232
    
    Also removes some (what I thought were unused) functions, and fixes a bug in clippy where we considered `gen fn` to be the same as `fn` because it was only built to consider asyncness.
    Zalathar authored Sep 12, 2024
    Configuration menu
    Copy the full SHA
    88fd5b7 View commit details
    Browse the repository at this point in the history
  9. Rollup merge of rust-lang#130256 - Zalathar:dump-stamp, r=jieyouxu

    Re-run coverage tests if `coverage-dump` was modified
    
    If the `coverage-dump` tool was modified, coverage tests should not be treated as up-to-date, because the tool's output might have changed.
    
    Bootstrap already handles rebuilding the tool itself if its sources were changed, so all compiletest needs to do here is include the binary in the list of files whose timestamps are checked.
    
    This should have no effect on non-coverage tests, because bootstrap won't pass the `--coverage-dump-path` flag, so the path in compiletest's config will be None.
    Zalathar authored Sep 12, 2024
    Configuration menu
    Copy the full SHA
    81cba5e View commit details
    Browse the repository at this point in the history