stable-sort temporary vector retention

Bug #496249 reported by Christophe Rhodes on 2009-12-13
This bug affects 1 person
Affects Status Importance Assigned to Milestone

Bug Description

Report from Keith James

 affects sbcl
 status confirmed
 importance medium
 tag sort performance space patch

>>>>> "Christophe" == Christophe Rhodes <email address hidden> writes:

    Christophe> In this day and age, I'm not sure
    Christophe> why stable-sort / vector-merge-sort doesn't simply
    Christophe> create a new vector for itself, which would at a
    Christophe> stroke eliminate both this leaks and worries about
    Christophe> threadsafety

I have modified my sbcl by removing *merge-sort-temp-vector* and
replacing it with a fresh vector each time. The result is that memory
use is much more manageable for stable-sorting vectors of big data.
The test suite runs successfully.

I've attached a patch for your consideration.

Changed in sbcl:
status: Confirmed → In Progress
Nikodemus Siivola (nikodemus) wrote :


Changed in sbcl:
status: In Progress → Fix Committed
Changed in sbcl:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers