zturner added a subscriber: zturner.

================
Comment at: source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp:2074
@@ +2073,3 @@
+
+                std::lock_guard<std::mutex> lock(index_mutex);
+                m_function_basename_index.Append(function_basename_index);
----------------
Every one of these is locking the same mutex.  You could make arrays outside of 
the async work that is `num_compile_units` entries, and put each result in its 
own entry in the array.

After the wait, you could make more async workers.  One for each variable.  
like `m_function_base_name_index.Append` could be one async job, and the same 
for each of the other ones.


http://reviews.llvm.org/D13662



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to