RFR: 8333133: Simplify QuickSort::sort [v2]
Kim Barrett
kbarrett at openjdk.org
Wed Jun 26 05:21:29 UTC 2024
On Wed, 19 Jun 2024 08:41:24 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> The "idempotent" argument is removed from that function, with associated
>> simplifications to the implementation. Callers are updated to remove that
>> argument. Callers that were providing a false value are unaffected in their
>> behavior. The 3 callers that were providing a true value to request the
>> associated feature are also unaffected (other than by being made faster),
>> because the arrays involved don't contain any equivalent pairs.
>>
>> There are also some miscellaneous cleanups, including using the swap utility
>> and fixing some comments.
>>
>> Testing: mach5 tier1-3
>
> Kim Barrett has updated the pull request incrementally with one additional commit since the last revision:
>
> improve find_pivot description
Thanks for reviews.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/19464#issuecomment-2190724041
More information about the shenandoah-dev
mailing list