diff options
| author | Nikita Popov <nikita.ppv@gmail.com> | 2020-02-16 17:46:26 +0100 |
|---|---|---|
| committer | Nikita Popov <nikita.ppv@gmail.com> | 2020-02-17 18:14:48 +0100 |
| commit | 80397d2d12b042586cb3bafdeb12ef8d982b8875 (patch) | |
| tree | c8ed10759fceee4be1b39f9771a6747e4eb883d7 /clang/lib/Serialization/ModuleManager.cpp | |
| parent | 98ed613cccd329d3892216b15d6e335f7c42a35b (diff) | |
| download | llvm-80397d2d12b042586cb3bafdeb12ef8d982b8875.zip llvm-80397d2d12b042586cb3bafdeb12ef8d982b8875.tar.gz llvm-80397d2d12b042586cb3bafdeb12ef8d982b8875.tar.bz2 | |
[IRBuilder] Delete copy constructor
D73835 will make IRBuilder no longer trivially copyable. This patch
deletes the copy constructor in advance, to separate out the breakage.
Currently, the IRBuilder copy constructor is usually used by accident,
not by intention. In rG7c362b25d7a9 I've fixed a number of cases where
functions accepted IRBuilder rather than IRBuilder &, thus performing
an unnecessary copy. In rG5f7b92b1b4d6 I've fixed cases where an
IRBuilder was copied, while an InsertPointGuard should have been used
instead.
The only non-trivial use of the copy constructor is the
getIRBForDbgInsertion() helper, for which I separated construction and
setting of the insertion point in this patch.
Differential Revision: https://reviews.llvm.org/D74693
Diffstat (limited to 'clang/lib/Serialization/ModuleManager.cpp')
0 files changed, 0 insertions, 0 deletions