On 19 Nov 2013, at 16:49, Sven Van Caekenberghe <[email protected]> wrote:

> 
> On 18 Nov 2013, at 23:12, Nicolas Cellier 
> <[email protected]> wrote:
> 
>> The tests pass with this hack PEGInfinity>>adaptToInteger: anInteger 
>> andCompare: selector 
>>    "This makes optimized inlining of timesRepeat: work."
>>    ^#( < <= ~= ) includes: selector
> 
> I asked Marcus for the way to do it, and if you add:
> 
> PEGParser class>>#compiler
>       ^ super compiler
>               options: #(- optionInlineTimesRepeat);
>               yourself
> 
> And do a
> 
> PEGParser recompile
> 
> All tests in Xtreams-ParsingTests should be green.
> 
>> But then they take 10 times longer than Squeak thanks to timesRepeat: 
>> inlining "optimization".
>> Maybe timesRepeat: inlining should be restricted to the case when receiver 
>> is an integer literal... Or just removed.
> 
> I am not sure I understand, that is more for the compiler guys.

I do not understand that sentence either, but I guess there is information 
missing.

        Marcus


Reply via email to