[llvm-commits] [Patch] Removal of Unwind from C-API

[David Blaikie]

> So I think the solution would be to properly document this on a webpage somewhere.

It should go in the release notes somewhere, I expect. (similar to
Chris's recent doc update for the Type system change:
http://llvm.org/viewvc/llvm-project/llvm/trunk/docs/ReleaseNotes.html?view=diff&r1=135375&r2=135376&pathrev=135376
)

> I would really rather not have dead code laying around...

Perhaps it's not so much "dead" code as historic code. Whether or not
it has the actual line commented out as well as an explanation, or
just has the explanation "An old feature was here & is now removed"
(or something like that) probably isn't too important.

Chances are as time goes on & these sort of changes crop up it'll be
more obvious why there are holes, even if they only have "skipping" as
your comment had.

Maybe simply: /* removed 6 due to API changes */ so someone's not left
wondering whether 6 is somehow special (as in currently
special/useful/etc).

- David