Re-enable multithreading in D version, use default configuration of vibe.d

nuald / simple-web-benchmark

A simple web benchmark of C++, Crystal, Go, Java, Node.js, PHP, Python, Rust and Scala.

The Unlicense

37 stars 9 forks source link

Re-enable multithreading in D version, use default configuration of vibe.d #11

Closed buggins closed 7 years ago

buggins commented 7 years ago

On my linux box, go version uses 14 threads (7 for 'go' process, and 7 for 'main' process). So, removing of multithreading from D test app looks strange.

P.S: please use 'dub upgrade --force-remove' after changing of dub.json to let dub choose new version if you already used dub with older version specified in dub.json

nuald commented 7 years ago

Multithreaded version fails on macOS, it just crashes with the "Program exited with code -11". I couldn't reproduce it without "--release" option, looks like some compiler optimizations have side effects. However, the brewed version of ldc2 is 1.3.0, I guess I have to try to use the latest version.

Anyway, I'll merge for now, but I think some changes will be needed anyway.