Remove unused IND_PERM
ClosedPublic

Authored by nomeata on Jan 22 2016, 4:16 AM.

Details

Summary

it seems that this closure type has not been in use since 5d52d9, so all
this is dead and untested code. This removes it. Some of the code might
be useful for a counting indirection as described in Trac #10613, so when
implementing that, have a look at what this commit removes.

Test Plan

validate on harbormaster

Diff Detail

Repository
rGHC Glasgow Haskell Compiler
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
nomeata updated this revision to Diff 6358.Jan 22 2016, 4:16 AM
nomeata retitled this revision from to Remove unused IND_PERM.
nomeata updated this object.
nomeata edited the test plan for this revision. (Show Details)
nomeata added a reviewer: simonmar.
bgamari accepted this revision.Jan 22 2016, 4:35 AM
bgamari edited edge metadata.

Looks reasonable to me but I'll defer judgment to @simonmar.

This revision is now accepted and ready to land.Jan 22 2016, 4:35 AM
simonmar accepted this revision.Jan 22 2016, 8:04 AM
simonmar edited edge metadata.

I don't mind removing this for now, but it might need to be resurrected in some form to fix Trac #5654

bgamari requested changes to this revision.Jan 22 2016, 9:47 AM
bgamari edited edge metadata.

I think you need to update ClosureFlags.c as well.

This revision now requires changes to proceed.Jan 22 2016, 9:47 AM
nomeata updated this revision to Diff 6374.Jan 22 2016, 3:02 PM
nomeata edited edge metadata.

Update sanity check in ClosureFlags.c

I think you need to update ClosureFlags.c as well.

This revision was automatically updated to reflect the committed changes.