diff options
author | David Blaikie <dblaikie@gmail.com> | 2015-03-03 23:53:03 +0000 |
---|---|---|
committer | David Blaikie <dblaikie@gmail.com> | 2015-03-03 23:53:03 +0000 |
commit | 19660f03bef13b883c9947d6c56805f62aa236b9 (patch) | |
tree | 2278f1926147f12d9545b3e7f391714feb06bfc9 /llvm/lib/CodeGen/LiveInterval.cpp | |
parent | 9e9e1d3ad2f8757a535ec6902b3240109e300f36 (diff) | |
download | bcm5719-llvm-19660f03bef13b883c9947d6c56805f62aa236b9.tar.gz bcm5719-llvm-19660f03bef13b883c9947d6c56805f62aa236b9.zip |
Recommit r231168: unique_ptrify LiveRange::segmentSet
GCC 4.7's libstdc++ doesn't have std::map::emplace, but it does have
std::unordered_map::emplace, and the use case here doesn't appear to
need ordering. The container has been changed in a separate/precursor
patch, and now this patch should hopefully build cleanly even with
GCC 4.7.
Original commit message:
This makes LiveRange non-copyable, and LiveInterval is already
non-movable (due to the explicit dtor), so now it's non-copyable and
non-movable.
Fix the one case where we were relying on the (deprecated in C++11)
implicit copy ctor of LiveInterval (which happened to work because the
ctor created an object with a null segmentSet, so double-deleting the
null pointer was fine).
llvm-svn: 231176
Diffstat (limited to 'llvm/lib/CodeGen/LiveInterval.cpp')
-rw-r--r-- | llvm/lib/CodeGen/LiveInterval.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/LiveInterval.cpp b/llvm/lib/CodeGen/LiveInterval.cpp index d60b0b1a504..e1aee4d898a 100644 --- a/llvm/lib/CodeGen/LiveInterval.cpp +++ b/llvm/lib/CodeGen/LiveInterval.cpp @@ -743,7 +743,6 @@ void LiveRange::flushSegmentSet() { segments.empty() && "segment set can be used only initially before switching to the array"); segments.append(segmentSet->begin(), segmentSet->end()); - delete segmentSet; segmentSet = nullptr; verify(); } |