Fix new clippy::unnecessary_map_or warnings.

Author: eddyb

crates/rustc_codegen_spirv/src/builder/builder_methods.rs

@@ -618,7 +618,7 @@ impl<'a, 'tcx> Builder<'a, 'tcx> {
 
             if offset == Size::ZERO
                 && leaf_size_range.contains(&ty_size)
-                && leaf_ty.map_or(true, |leaf_ty| leaf_ty == ty)
+                && leaf_ty.is_none_or(|leaf_ty| leaf_ty == ty)
             {
                 trace!("successful recovery leaf type: {:?}", self.debug_type(ty));
                 trace!("successful recovery indices: {:?}", indices);

crates/rustc_codegen_spirv/src/linker/import_export_link.rs

@@ -229,10 +229,9 @@ fn kill_linkage_instructions(
         .retain(|f| !rewrite_rules.contains_key(&f.def_id().unwrap()));
 
     // drop imported variables
-    module.types_global_values.retain(|v| {
-        v.result_id
-            .map_or(true, |v| !rewrite_rules.contains_key(&v))
-    });
+    module
+        .types_global_values
+        .retain(|v| v.result_id.is_none_or(|v| !rewrite_rules.contains_key(&v)));
 
     // NOTE(eddyb) `Options`'s `keep_link_export`s field requests that `Export`s
     // are left in (primarily for unit testing - see also its doc comment).
@@ -264,21 +263,21 @@ fn import_kill_annotations_and_debug(
 ) {
     module.annotations.retain(|inst| {
         inst.operands.is_empty()
-            || inst.operands[0].id_ref_any().map_or(true, |id| {
+            || inst.operands[0].id_ref_any().is_none_or(|id| {
                 !rewrite_rules.contains_key(&id) && !killed_parameters.contains(&id)
             })
     });
     module.debug_names.retain(|inst| {
         inst.operands.is_empty()
-            || inst.operands[0].id_ref_any().map_or(true, |id| {
+            || inst.operands[0].id_ref_any().is_none_or(|id| {
                 !rewrite_rules.contains_key(&id) && !killed_parameters.contains(&id)
             })
     });
     // need to remove OpGroupDecorate members that mention this id
     for inst in &mut module.annotations {
         if inst.class.opcode == Op::GroupDecorate {
             inst.operands.retain(|op| {
-                op.id_ref_any().map_or(true, |id| {
+                op.id_ref_any().is_none_or(|id| {
                     !rewrite_rules.contains_key(&id) && !killed_parameters.contains(&id)
                 })
             });

crates/rustc_codegen_spirv/src/linker/mem2reg.rs

@@ -598,7 +598,7 @@ fn remove_old_variables(
         block.instructions.retain(|inst| {
             !matches!(inst.class.opcode, Op::AccessChain | Op::InBoundsAccessChain)
                 || inst.operands.iter().all(|op| {
-                    op.id_ref_any().map_or(true, |id| {
+                    op.id_ref_any().is_none_or(|id| {
                         var_maps_and_types
                             .iter()
                             .all(|(var_map, _)| !var_map.contains_key(&id))