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).

hercules_opt/src/delete_uncalled.rs

@@ -50,6 +50,11 @@ pub fn delete_uncalled(
     // that all nodes in all functions will be mutable, so panic if any
     // edit fails.
     for editor in editors.iter_mut() {
+        // Don't make edits to dead functions as they may include calls to dead functions
+        if new_idx[editor.func_id().idx()].is_none() {
+            continue;
+        }
+
         let callsites: Vec<_> = editor
             .node_ids()
             .filter(|id| editor.func().nodes[id.idx()].is_call())