diff options
| author | Mark de Wever <koraq@xs4all.nl> | 2020-01-11 10:16:40 +0100 |
|---|---|---|
| committer | Mark de Wever <koraq@xs4all.nl> | 2020-01-11 15:34:02 +0100 |
| commit | 9c74fb402e1b7aad4a509a49ab4792154b8ba2c8 (patch) | |
| tree | 132e905be5a2a736ac7f1a9f6944e6b4e6d14eb6 /llvm/lib/CodeGen/MachineInstr.cpp | |
| parent | 08275a52d83e623f0347fd9396c18f4d21a15c90 (diff) | |
| download | bcm5719-llvm-9c74fb402e1b7aad4a509a49ab4792154b8ba2c8.tar.gz bcm5719-llvm-9c74fb402e1b7aad4a509a49ab4792154b8ba2c8.zip | |
[Sema] Improve -Wrange-loop-analysis warnings.
No longer generate a diagnostic when a small trivially copyable type is
used without a reference. Before the test looked for a POD type and had no
size restriction. Since the range-based for loop is only available in
C++11 and POD types are trivially copyable in C++11 it's not required to
test for a POD type.
Since copying a large object will be expensive its size has been
restricted. 64 bytes is a common size of a cache line and if the object is
aligned the copy will be cheap. No performance impact testing has been
done.
Differential Revision: https://reviews.llvm.org/D72212
Diffstat (limited to 'llvm/lib/CodeGen/MachineInstr.cpp')
0 files changed, 0 insertions, 0 deletions
