Wanted: Review manager for std.data.json

[John Colvin via Digitalmars-d]

On Thursday, 9 April 2015 at 19:40:31 UTC, Sönke Ludwig wrote:
> Am 09.04.2015 um 21:37 schrieb weaselcat:
>> On Thursday, 9 April 2015 at 19:35:24 UTC, Sönke Ludwig wrote:
>>> Am 09.04.2015 um 21:26 schrieb weaselcat:
>>>> On Thursday, 9 April 2015 at 19:17:48 UTC, Sönke Ludwig 
>>>> wrote:
>>>>
>>>>> Not sure, but that may also have been my recent 
>>>>> optimizations.
>>>>
>>>> Just tried it with your recent optimizations and it doesn't 
>>>> build with
>>>> LDC 0.15.1.
>>>
>>> Should work now. I just tested LDC with -singleobj and it's 
>>> faster
>>> than RapidJSON! Now the DOM parser gets interesting.
>>
>> What flags are you using? RapidJSON is still a lot faster for 
>> me.
>> also, I just realized the build script has the wrong inline 
>> flag for
>> LDC(it's using -enable-inlining instead of -inline)
>
> I'll have to test with that exact version, because I've used my 
> own main() function for the test. The flags were: -singleobj 
> -release -O5 -enable-inlining -disable-bondscheck 
> (-disable-boundscheck is only a little faster).

I can't remember which -O level inlining is enabled, but there's 
definitely no need to explicitly ask for it at -O5.