[GHC] #13908: sameMutableArray equivalent for Array
GHC
ghc-devs at haskell.org
Sat Jul 1 14:47:37 UTC 2017
#13908: sameMutableArray equivalent for Array
-------------------------------------+-------------------------------------
Reporter: winter | Owner: (none)
Type: bug | Status: new
Priority: normal | Milestone:
Component: Compiler | Version: 8.0.1
Keywords: | Operating System: Unknown/Multiple
Architecture: | Type of failure: None/Unknown
Unknown/Multiple |
Test Case: | Blocked By:
Blocking: | Related Tickets:
Differential Rev(s): | Wiki Page:
-------------------------------------+-------------------------------------
Sometime it is useful to compare reference equality for immutable arrays,
for example when comparing vectors which are slices of arrays, we may want
to know if two vectors are slices of the same array. But currently we only
have `sameMutableXXXArray` primitives.
A workaround may look like this:
{{{
sameArray :: Array a -> Array a -> Bool
sameArray arr1 arr2 = runST (do
marr1 <- unsafeThawArray arr1
marr2 <- unsafeThawArray arr2
return (sameMutableArray marr1 marr2)
)
}}}
But the problem is for boxed arrays, this code will push an unchanged
array to mutable list if it's already on older generations. Next GC will
have to scan it(at least will scan its card table).
I can see two solutions here, first one is to add `sameXXXArray` for all
array types. The second one is somehow more complex:
Currently we simply rewrite array's header to
`MUT_ARR_PTRS_FROZEN0/SMALL_MUT_ARR_PTRS_FROZEN0` when we do
`unsafeFreeze`. But for mutable array with no write happend, e.g. the ones
with `MUT_ARR_PTRS_CLEAN/SMALL_MUT_ARR_PTRS_CLEAN` header. we can safely
rewrite the header to `MUT_ARR_PTRS_FROZEN/SMALL_MUT_ARR_PTRS_FROZEN`, so
that we can keep it from next GC.
Then to fix previous code, we can add a freeze before returning.
Of course we can do these two solutions all together ; )
--
Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/13908>
GHC <http://www.haskell.org/ghc/>
The Glasgow Haskell Compiler
More information about the ghc-tickets
mailing list