eliminate ValueSymbolTable::rename, it has no advantage over using remove+insert.

Make insert/remove assert if used incorrectly instead of returning a bool.

llvm-svn: 33988