[clang] [clang] Mark -Wlarge-by-value-copy diagnostics DefaultIgnore (PR #214696)

[via cfe-commits]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: richardmilles

<details>
<summary>Changes</summary>

These warnings only fire when a size threshold is provided, so they should not be documented as enabled by default.

Fixes #<!-- -->37523


---
Full diff: https://github.com/llvm/llvm-project/pull/214696.diff


1 Files Affected:

- (modified) clang/include/clang/Basic/DiagnosticSemaKinds.td (+2-2) 


``````````diff
diff --git a/clang/include/clang/Basic/DiagnosticSemaKinds.td b/clang/include/clang/Basic/DiagnosticSemaKinds.td
index b57ff6c197d1d..738537ba40c91 100644
--- a/clang/include/clang/Basic/DiagnosticSemaKinds.td
+++ b/clang/include/clang/Basic/DiagnosticSemaKinds.td
@@ -548,10 +548,10 @@ def warn_import_implementation_partition_unit_in_interface_unit : Warning<
 
 def warn_parameter_size: Warning<
   "%0 is a large (%1 bytes) pass-by-value argument; "
-  "pass it by reference instead ?">, InGroup<LargeByValueCopy>;
+  "pass it by reference instead ?">, InGroup<LargeByValueCopy>, DefaultIgnore;
 def warn_return_value_size: Warning<
   "return value of %0 is a large (%1 bytes) pass-by-value object; "
-  "pass it by reference instead ?">, InGroup<LargeByValueCopy>;
+  "pass it by reference instead ?">, InGroup<LargeByValueCopy>, DefaultIgnore;
 def warn_return_value_udt: Warning<
   "%0 has C-linkage specified, but returns user-defined type %1 which is "
   "incompatible with C">, InGroup<ReturnTypeCLinkage>;

``````````

</details>


https://github.com/llvm/llvm-project/pull/214696