Wanted: Review manager for std.data.json

[weaselcat via Digitalmars-d]

On Thursday, 9 April 2015 at 21:52:08 UTC, Sönke Ludwig wrote:
> Am 09.04.2015 um 23:14 schrieb weaselcat:
>> On Thursday, 9 April 2015 at 20:11:07 UTC, weaselcat wrote:
>>> On Thursday, 9 April 2015 at 20:00:28 UTC, John Colvin wrote:
>>>> I can't remember which -O level inlining is enabled, but 
>>>> there's
>>>> definitely no need to explicitly ask for it at -O5.
>>>
>>> -enable-inlining(enabled at -O2) and -inline do different 
>>> things and
>>> -inline isn't automatically enabled.
>>
>> Sorry Sönke for polluting your thread, but I did some reading 
>> of the
>> LLVM manual - -inline is a specific LLVM pass and not a 
>> frontend flag.
>> Using the -partial-inliner pass instead it(the benchmark) 
>> becomes faster
>> than rapidJSON.
>>
>> Bye.
>
> Hm interesting, but for me LDC unfortunately crashes when 
> adding -partial-inliner. What else did you specify?

ldc2 -partial-inliner -boundscheck=off -singleobj 
-ofjson_d_new_lazy_ldc -O5 -release -w -oq 
-Istd_data_json/source/ std_data_json/source/stdx/data/json/*.d 
test_new_lazy.d

it seems to segfault with -O0 optimization

./json_d_new_lazy_ldc  0.95s user 0.08s system 99% cpu 1.025 total
(clang)./json_rapid_cpp  0.98s user 0.30s system 99% cpu 1.282 
total
(gcc)./json_rapid_cpp  0.80s user 0.31s system 99% cpu 1.108 total