New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Apply rules concurrently #44
Comments
I've just played around with this a bit, but I was only able to get modest gains by parallelizing the rules. I think it would be a lot more worthwhile to have one thread per file (including compilation) rather than per rule. Experiment resultsMacBook Pro 4x2.4 GHz (late 2011) linear
rules parallel (8 threads)
files parallel (8 threads)
|
Interestingly, dogfooding.sh actually fared much better with 2:43, 1:37, 1:16 wall time... |
@nschum, your observation is quite right. Currently, the performance bottleneck is the compilation time rather than the analysis time (unless the users have a huge rule set, but this is hardly true for now). But in the future, when we have more rules and certain rules will require heavy analysis, so eventually the analysis time will be prominent. I have taken a look at your pull request briefly, as always, very impressive quality code in high standard. Thank you for the help. I will get back to you in detail later. |
We would like to have the option to apply rules concurrently against translation units. This should be able to improve the performance largely.
The text was updated successfully, but these errors were encountered: