46e38f36785508dc6f90e642c68e4a72a493e8f5 - toolchain/llvm-project

commit	46e38f36785508dc6f90e642c68e4a72a493e8f5	[log] [tgz]
author	Benjamin Kramer <benny.kra@googlemail.com>	Wed Jun 08 10:01:20 2016 +0000
committer	Benjamin Kramer <benny.kra@googlemail.com>	Wed Jun 08 10:01:20 2016 +0000
tree	290f0903a81451736d6c920ef4b50bba0e995325
parent	acadc8e0d64e90cded8f63a154ab3111a1d294f9 [diff]

Avoid copies of std::strings and APInt/APFloats where we only read from it

As suggested by clang-tidy's performance-unnecessary-copy-initialization.
This can easily hit lifetime issues, so I audited every change and ran the
tests under asan, which came back clean.

llvm-svn: 272126

llvm/lib/Analysis/DependenceAnalysis.cpp[diff]
llvm/lib/Analysis/InstructionSimplify.cpp[diff]
llvm/lib/Analysis/ValueTracking.cpp[diff]
llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp[diff]
llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp[diff]
llvm/lib/IR/Verifier.cpp[diff]
llvm/lib/TableGen/Record.cpp[diff]
llvm/lib/Target/AArch64/AArch64ISelLowering.cpp[diff]
llvm/lib/Target/ARM/ARMISelLowering.cpp[diff]
llvm/lib/Target/Hexagon/BitTracker.cpp[diff]
llvm/lib/Target/Hexagon/HexagonISelDAGToDAG.cpp[diff]
llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp[diff]
llvm/lib/Target/NVPTX/NVPTXISelLowering.cpp[diff]
llvm/lib/Target/X86/X86ISelLowering.cpp[diff]
llvm/lib/Target/X86/X86MCInstLower.cpp[diff]
llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp[diff]
llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp[diff]
llvm/lib/Transforms/InstCombine/InstCombineSelect.cpp[diff]
llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp[diff]
llvm/lib/Transforms/Scalar/Float2Int.cpp[diff]
llvm/lib/Transforms/Scalar/GuardWidening.cpp[diff]
llvm/lib/Transforms/Scalar/Reassociate.cpp[diff]
llvm/tools/lli/lli.cpp[diff]

23 files changed

tree: 290f0903a81451736d6c920ef4b50bba0e995325

clang/
clang-tools-extra/
compiler-rt/
debuginfo-tests/
libclc/
libcxx/
libcxxabi/
libunwind/
lld/
lldb/
llgo/
llvm/
openmp/
polly/