linker/inline: also run remove_duplicate_debuginfo on every fully-inlined function.

Author: eddyb

crates/rustc_codegen_spirv/src/linker/duplicates.rs

@@ -283,7 +283,20 @@ pub fn remove_duplicate_debuginfo(module: &mut Module) {
         })
         .map(|inst| inst.result_id.unwrap());
 
+    let deduper = DebuginfoDeduplicator {
+        custom_ext_inst_set_import,
+    };
     for func in &mut module.functions {
+        deduper.remove_duplicate_debuginfo_in_function(func)
+    }
+}
+
+pub struct DebuginfoDeduplicator {
+    pub custom_ext_inst_set_import: Option<Word>,
+}
+
+impl DebuginfoDeduplicator {
+    pub fn remove_duplicate_debuginfo_in_function(&self, func: &mut rspirv::dr::Function) {
         for block in &mut func.blocks {
             // Ignore the terminator, it's effectively "outside" debuginfo.
             let (_, insts) = block.instructions.split_last_mut().unwrap();
@@ -339,7 +352,8 @@ pub fn remove_duplicate_debuginfo(module: &mut Module) {
                 let inst = &insts[inst_idx];
                 let custom_op = match inst.class.opcode {
                     Op::ExtInst
-                        if Some(inst.operands[0].unwrap_id_ref()) == custom_ext_inst_set_import =>
+                        if Some(inst.operands[0].unwrap_id_ref())
+                            == self.custom_ext_inst_set_import =>
                     {
                         Some(CustomOp::decode_from_ext_inst(inst))
                     }

crates/rustc_codegen_spirv/src/linker/inline.rs

@@ -139,6 +139,11 @@ pub fn inline(sess: &Session, module: &mut Module) -> super::Result<()> {
         let mut function = mem::replace(&mut functions[func_idx], Err(FuncIsBeingInlined)).unwrap();
         inliner.inline_fn(&mut function, &functions);
         fuse_trivial_branches(&mut function);
+
+        super::duplicates::DebuginfoDeduplicator {
+            custom_ext_inst_set_import,
+        }.remove_duplicate_debuginfo_in_function(&mut function);
+
         functions[func_idx] = Ok(function);
     }