trivially_copy_pass_by_ref: print size of type and limit in the lint …

…message

clippy_lints/src/trivially_copy_pass_by_ref.rs

@@ -65,7 +65,7 @@ impl<'a, 'tcx> TriviallyCopyPassByRef {
             // portability problems between 32 and 64-bit targets
             let bit_width = cmp::min(bit_width, 32);
             let byte_width = bit_width / 8;
-            // Use a limit of 2 times the register bit width
+            // Use a limit of 2 times the register byte width
             byte_width * 2
         });
         Self { limit }
@@ -118,7 +118,7 @@ impl<'a, 'tcx> TriviallyCopyPassByRef {
                         cx,
                         TRIVIALLY_COPY_PASS_BY_REF,
                         input.span,
-                        "this argument is passed by reference, but would be more efficient if passed by value",
+                        &format!("this argument ({} B) is passed by reference, but would be more efficient if passed by value (limit: {} B)", size, self.limit),
                         "consider passing by value instead",
                         value_type,
                         Applicability::Unspecified,

tests/ui/trivially_copy_pass_by_ref.stderr

@@ -1,90 +1,90 @@
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:47:11
    |
 LL | fn bad(x: &u32, y: &Foo, z: &Baz) {}
    |           ^^^^ help: consider passing by value instead: `u32`
    |
    = note: `-D clippy::trivially-copy-pass-by-ref` implied by `-D warnings`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:47:20
    |
 LL | fn bad(x: &u32, y: &Foo, z: &Baz) {}
    |                    ^^^^ help: consider passing by value instead: `Foo`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:47:29
    |
 LL | fn bad(x: &u32, y: &Foo, z: &Baz) {}
    |                             ^^^^ help: consider passing by value instead: `Baz`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:54:12
    |
 LL |     fn bad(&self, x: &u32, y: &Foo, z: &Baz) {}
    |            ^^^^^ help: consider passing by value instead: `self`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:54:22
    |
 LL |     fn bad(&self, x: &u32, y: &Foo, z: &Baz) {}
    |                      ^^^^ help: consider passing by value instead: `u32`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:54:31
    |
 LL |     fn bad(&self, x: &u32, y: &Foo, z: &Baz) {}
    |                               ^^^^ help: consider passing by value instead: `Foo`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:54:40
    |
 LL |     fn bad(&self, x: &u32, y: &Foo, z: &Baz) {}
    |                                        ^^^^ help: consider passing by value instead: `Baz`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:56:16
    |
 LL |     fn bad2(x: &u32, y: &Foo, z: &Baz) {}
    |                ^^^^ help: consider passing by value instead: `u32`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:56:25
    |
 LL |     fn bad2(x: &u32, y: &Foo, z: &Baz) {}
    |                         ^^^^ help: consider passing by value instead: `Foo`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:56:34
    |
 LL |     fn bad2(x: &u32, y: &Foo, z: &Baz) {}
    |                                  ^^^^ help: consider passing by value instead: `Baz`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:68:16
    |
 LL |     fn bad2(x: &u32, y: &Foo, z: &Baz) {}
    |                ^^^^ help: consider passing by value instead: `u32`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:68:25
    |
 LL |     fn bad2(x: &u32, y: &Foo, z: &Baz) {}
    |                         ^^^^ help: consider passing by value instead: `Foo`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:68:34
    |
 LL |     fn bad2(x: &u32, y: &Foo, z: &Baz) {}
    |                                  ^^^^ help: consider passing by value instead: `Baz`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:72:34
    |
 LL |     fn trait_method(&self, _foo: &Foo);
    |                                  ^^^^ help: consider passing by value instead: `Foo`
 
-error: this argument is passed by reference, but would be more efficient if passed by value
+error: this argument (4 B) is passed by reference, but would be more efficient if passed by value (limit: 8 B)
   --> $DIR/trivially_copy_pass_by_ref.rs:76:37
    |
 LL |     fn trait_method2(&self, _color: &Color);