[PATCH] D12605: [ELF2] Don't allocate more SymbolBodies than needed.

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 3 13:14:09 PDT 2015


davide created this revision.
davide added reviewers: rafael, Bigcheese.
davide added a subscriber: llvm-commits.

We're aiming  at not wasting space, but let's try to do it properly (hopefully).

http://reviews.llvm.org/D12605

Files:
  ELF/InputFiles.cpp

Index: ELF/InputFiles.cpp
===================================================================
--- ELF/InputFiles.cpp
+++ ELF/InputFiles.cpp
@@ -110,7 +110,7 @@
   if (FirstNonLocal > NumSymbols)
     error("Invalid sh_info in symbol table");
   Syms = llvm::make_range(Syms.begin() + FirstNonLocal, Syms.end());
-  SymbolBodies.reserve(NumSymbols);
+  SymbolBodies.reserve(NumSymbols - FirstNonLocal);
   for (const Elf_Sym &Sym : Syms)
     SymbolBodies.push_back(createSymbolBody(StringTable, &Sym));
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12605.33966.patch
Type: text/x-patch
Size: 506 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150903/556cd673/attachment.bin>


More information about the llvm-commits mailing list