[Lldb-commits] [lldb] [lldb][Symtab][NFCI] Replace vector::swap with shrink_to_fit (PR #70918)

Michael Buch via lldb-commits lldb-commits at lists.llvm.org
Wed Nov 1 03:31:28 PDT 2023


https://github.com/Michael137 created https://github.com/llvm/llvm-project/pull/70918

Replaces the old idiom of swapping the container to shrink it with the newer STL alternative.

Similar transition in LLDB was done in: https://reviews.llvm.org/D47492

>From 52c5caf0c352fc13377e56367cc6935e3d9c8c3f Mon Sep 17 00:00:00 2001
From: Michael Buch <michaelbuch12 at gmail.com>
Date: Wed, 1 Nov 2023 10:29:20 +0000
Subject: [PATCH] [lldb][Symtab][NFCI] Replace vector::swap with shrink_to_fit

---
 lldb/source/Symbol/Symtab.cpp | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/lldb/source/Symbol/Symtab.cpp b/lldb/source/Symbol/Symtab.cpp
index 104faac38ffa0fe..1aebe198f9e78ae 100644
--- a/lldb/source/Symbol/Symtab.cpp
+++ b/lldb/source/Symbol/Symtab.cpp
@@ -1010,10 +1010,7 @@ void Symtab::Finalize() {
   // Calculate the size of symbols inside InitAddressIndexes.
   InitAddressIndexes();
   // Shrink to fit the symbols so we don't waste memory
-  if (m_symbols.capacity() > m_symbols.size()) {
-    collection new_symbols(m_symbols.begin(), m_symbols.end());
-    m_symbols.swap(new_symbols);
-  }
+  m_symbols.shrink_to_fit();
   SaveToCache();
 }
 



More information about the lldb-commits mailing list