More notes on why uninteruptible makes sense

src/General/Cleanup.hs

@@ -36,6 +36,7 @@ newCleanup = do
     -- important to use uninterruptibleMask_ otherwise in things like allocateThread
     -- we might end up being interrupted and failing to close down the thread
     -- e.g. see https://github.com/digital-asset/ghcide/issues/381
+    -- note that packages like safe-exceptions also use uninterruptibleMask_
     let clean = uninterruptibleMask_ $ do
             items <- atomicModifyIORef' ref $ \s -> (s{items=Map.empty}, items s)
             mapM_ snd $ sortOn (negate . fst) $ Map.toList items