Re: Re: Faster zend sorting implementation

From: Date: Wed, 14 Jan 2015 20:09:41 +0000
Subject: Re: Re: Faster zend sorting implementation
References: 1 2  Groups: php.internals 
Request: Send a blank email to [email protected] to get a copy of this message
Hi!

>     I made a PR here: https://github.com/php/php-src/pull/999 for
> reviewing
> 
>     in benchmark this can brings more than 30% performance gain in
> array_sort etc functions.
> 
>     tests fails are related to non-stable vs stable sorting difference.
> 
>     anyway, I feel it's better to ask you to do a final review, what
> do you think?
> 
>     is there any objections to merge this?

I think the sort order of equal elements was never defined, so changing
it would not be a big issue. The tests, of course, need to be fixed and
note in UPGRADING should be provided, but otherwise it's fine.

-- 
Stas Malyshev
[email protected]


Thread (23 messages)

« previous php.internals (#80486) next »