stable-sort temporary vector retention

Bug #496249 reported by Christophe Rhodes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
SBCL
Fix Released
Medium
Unassigned

Bug Description

Report from Keith James

 affects sbcl
 status confirmed
 importance medium
 tag sort performance space patch
 done

>>>>> "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.

Revision history for this message
Christophe Rhodes (csr21-cantab) wrote :
Changed in sbcl:
status: Confirmed → In Progress
Revision history for this message
Nikodemus Siivola (nikodemus) wrote :

In SBCL 1.0.36.18.

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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.