[binutils-gdb] Don't erase empty indices in DWARF reader

[binutils-gdb] Don't erase empty indices in DWARF reader

[Tom Tromey]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=1a6191f1a614f61d20bfe414b81c39a8ca785cc7

commit 1a6191f1a614f61d20bfe414b81c39a8ca785cc7
Author: Tom Tromey <tromey@adacore.com>
Date:   Wed Dec 14 07:20:49 2022 -0700

    Don't erase empty indices in DWARF reader
    
    The DWARF reader has some code to remove empty indices.  However, I
    think this code has been obsolete since some earlier changes to
    parallel_for_each.  This patch removes this code.

Diff:
---
 gdb/dwarf2/read.c | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c
index 851852404eb..c3f246fedf7 100644
--- a/gdb/dwarf2/read.c
+++ b/gdb/dwarf2/read.c
@@ -7170,16 +7170,6 @@ dwarf2_build_psymtabs_hard (dwarf2_per_objfile *per_objfile)
     print_tu_stats (per_objfile);
 
   indexes.push_back (index_storage.release ());
-  /* Remove any NULL entries.  This might happen if parallel-for
-     decides to throttle the number of threads that were used.  */
-  indexes.erase
-    (std::remove_if (indexes.begin (),
-		     indexes.end (),
-		     [] (const std::unique_ptr<cooked_index> &entry)
-		     {
-		       return entry == nullptr;
-		     }),
-     indexes.end ());
   indexes.shrink_to_fit ();
 
   cooked_index_vector *vec = new cooked_index_vector (std::move (indexes));