[PATCH] D30691: [analyzer] Support for naive cross translational unit analysis

Daniel Marjamäki via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 3 00:22:48 PDT 2017


danielmarjamaki added inline comments.


================
Comment at: tools/xtu-analysis/xtu-analyze.py:29
+
+threading_factor = int(multiprocessing.cpu_count() * 1.5)
+analyser_output_formats = ['plist-multi-file', 'plist', 'plist-html',
----------------
gerazo wrote:
> danielmarjamaki wrote:
> > does this mean that if there are 4 cores this script will by default use 6 threads? isn't that too aggressive?
> Yes, it does mean that. You are right, it is aggressive. To be honest, the xtu-build step is more io intensive where it really makes sense. In the xtu-analyze step, it is marginal when big files are compiled (more cpu, less io).  We will put this one back to 1.0 instead.
I see. Feel free to use such ratio.


https://reviews.llvm.org/D30691





More information about the cfe-commits mailing list