Make the iterator form of erase return void, since it always succeeds,

and since this is what std::map and std::set do.

llvm-svn: 112701