https://issues.dlang.org/show_bug.cgi?id=16517
Andrei Alexandrescu <and...@erdani.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |and...@erdani.com
--- Comment #1 from Andrei Alexandrescu <and...@erdani.com> ---
I don't have a ldc testing rig installed, but with
https://github.com/dlang/phobos/pull/4815 and dmd I get the following for your
code and data:
$ dmd -O -inline -release -ofmedian_by_topn -boundscheck=off -version=topn
issue16517.d
$ dmd -O -inline -release -ofmedian_by_sort -boundscheck=off issue16517.d
$ cut -f 2 /tmp/googlebooks-eng-all-1gram-20120701-0 | ./median_by_sort
median (via sort): 1972; lines: 10512769; total time (ms): 2246; sort time
(ms): 412
$ cut -f 2 /tmp/googlebooks-eng-all-1gram-20120701-0 | ./median_by_topn
median (via topN): 1972; lines: 10512769; total time (ms): 1823; topN time
(ms): 35
Although results obtained using dmd are not comparable, the trend is
encouraging. If you could patch your stdlib with the pull request it would be
great. Thanks!
--
