[Lldb-commits] [PATCH] D32598: Prime module caches outside of the module list lock for better parallelism

[Scott Smith via Phabricator via lldb-commits]

scott.smith added a comment.

Here's the controversial patch.  It has been brought up that the intent is to not load symbols before they are needed, but at least in my experience this patch has no effect on performance when running:
lldb -b -o run /path/to/myprogram
where myprogram has main(){return 0;} and links in a whole lot of libraries.  My platform is Ubuntu 14.04; are there other systems where symbol processing is actually delayed?  Maybe on those systems PrimeCaches() can be a no-op to not impact performance?

This patch requires https://reviews.llvm.org/D32597 in order to actually improve performance, but does not require it for correctness.


Repository:
  rL LLVM

https://reviews.llvm.org/D32598