about summary refs log tree commit diff
path: root/compiler/rustc_codegen_llvm/src
diff options
context:
space:
mode:
authorMatthias Krüger <matthias.krueger@famsik.de>2024-02-24 15:35:15 +0100
committerGitHub <noreply@github.com>2024-02-24 15:35:15 +0100
commit7e8fa8fb24b9d89402ba7ce6cb3156cb3da4b1a3 (patch)
tree3cce081f3e087371bf99978553fa3eba8b907800 /compiler/rustc_codegen_llvm/src
parentd978a1c671a4c6fb61e96bad392558ba750e0d27 (diff)
parentff930d4fed085dcff75c7b9df6af3a4e099d2f21 (diff)
downloadrust-7e8fa8fb24b9d89402ba7ce6cb3156cb3da4b1a3.tar.gz
rust-7e8fa8fb24b9d89402ba7ce6cb3156cb3da4b1a3.zip
Rollup merge of #121539 - Enselic:fix-large-move, r=Nilstrieb
compiler/rustc_target/src/spec/base/apple/tests.rs: Avoid unnecessary large move

Fixes:

    $ MAGIC_EXTRA_RUSTFLAGS=-Zmove-size-limit=4096 ./x test compiler/rustc_target
    error: moving 6216 bytes
      --> compiler/rustc_target/src/spec/base/apple/tests.rs:17:19
       |
    17 |     for target in all_sim_targets {
       |                   ^^^^^^^^^^^^^^^ value moved from here
       |
       = note: The current maximum size is 4096, but it can be customized with the move_size_limit attribute: `#![move_size_limit = "..."]`
       = note: `-D large-assignments` implied by `-D warnings`
       = help: to override `-D warnings` add `#[allow(large_assignments)]`

Part of https://github.com/rust-lang/rust/issues/83518

The goal is to enable the lint by default at one point. There are many cases where it needs to be allowed. But here we don't need to allow it. We can simply fix it.

I don't want to enable the lint in the file, because I don't want the "default size threshold for when a value is OK to move" to be duplicated in many places. We'll want the threshold in one central place once we enable it by default.
Diffstat (limited to 'compiler/rustc_codegen_llvm/src')
0 files changed, 0 insertions, 0 deletions