Skip to content
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

Run for module generation in parallel #193

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wilzbach
Copy link
Member

@wilzbach wilzbach commented Jan 15, 2018

Currently dpl-docs on dlang.org run a single-thread. That makes them the bottleneck in time consumption.

@s-ludwig
Copy link
Member

s-ludwig commented Sep 2, 2020

Unfortunately this change causes multiple race conditions. I think when we make an attempt to parallelize document generation, we should also use pure and shared/immutable within the affected call tree to let the compiler make reasonably sure that the approach is correct. Frankly, I don't feel comfortable to do multi-threading these days without either that or guaranteed memory isolation, unless it is a small and isolated piece of code. Too many bad memories from C++ times ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants