From b096524827460d2a3a7e7d08ae6c2f41a1388bab Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Sun, 15 Oct 2023 12:15:36 -0600 Subject: Use try_emplace in index-write.c index-write.c has a comment indicating that C++17's try_emplace could be used. This patch makes the change. Approved-By: Pedro Alves --- gdb/dwarf2/index-write.c | 25 +++++++++---------------- 1 file changed, 9 insertions(+), 16 deletions(-) (limited to 'gdb/dwarf2') diff --git a/gdb/dwarf2/index-write.c b/gdb/dwarf2/index-write.c index b77f4f9..b4a0117 100644 --- a/gdb/dwarf2/index-write.c +++ b/gdb/dwarf2/index-write.c @@ -515,24 +515,17 @@ write_hash_table (mapped_symtab *symtab, data_buf &output, data_buf &cpool) continue; gdb_assert (entry.index_offset == 0); - /* Finding before inserting is faster than always trying to - insert, because inserting always allocates a node, does the - lookup, and then destroys the new node if another node - already had the same key. C++17 try_emplace will avoid - this. */ - const auto found - = symbol_hash_table.find (entry.cu_indices); - if (found != symbol_hash_table.end ()) + auto [iter, inserted] + = symbol_hash_table.try_emplace (entry.cu_indices, + cpool.size ()); + entry.index_offset = iter->second; + if (inserted) { - entry.index_offset = found->second; - continue; + /* Newly inserted. */ + cpool.append_offset (entry.cu_indices.size ()); + for (const auto index : entry.cu_indices) + cpool.append_offset (index); } - - symbol_hash_table.emplace (entry.cu_indices, cpool.size ()); - entry.index_offset = cpool.size (); - cpool.append_offset (entry.cu_indices.size ()); - for (const auto index : entry.cu_indices) - cpool.append_offset (index); } } -- cgit v1.1