Fix delete uncalled bugs

Fixes two bugs related to delete uncalled:

1. Functions which are going to be deleted should not be edited to update the call sites since they main contain calls to functions that are also going to be deleted and so have no new Function ID.
2. Fix to updating the pass manager's environment; the environment's map contain variables with no bindings and hence no associated scopes (this can arise from running delete uncalled twice but also from variables just going out of scope).

requested review from @rarbore2

assigned to @aaronjc4

mentioned in commit 1aad2842

merged