diff --git a/clippy_dev/src/ra_setup.rs b/clippy_dev/src/ra_setup.rs
index 40bf4a9505a8..39538e632b11 100644
--- a/clippy_dev/src/ra_setup.rs
+++ b/clippy_dev/src/ra_setup.rs
@@ -1,5 +1,3 @@
-#![allow(clippy::filter_map)]
-
 use std::fs;
 use std::fs::File;
 use std::io::prelude::*;
diff --git a/clippy_lints/src/loops.rs b/clippy_lints/src/loops.rs
index 1bd96b2b4c89..d566bd8ca3ea 100644
--- a/clippy_lints/src/loops.rs
+++ b/clippy_lints/src/loops.rs
@@ -1061,7 +1061,6 @@ fn get_assignments<'a: 'c, 'tcx: 'c, 'c>(
 ) -> impl Iterator<Item = Option<(&'tcx Expr<'tcx>, &'tcx Expr<'tcx>)>> + 'c {
     // As the `filter` and `map` below do different things, I think putting together
     // just increases complexity. (cc #3188 and #4193)
-    #[allow(clippy::filter_map)]
     stmts
         .iter()
         .filter_map(move |stmt| match stmt.kind {
diff --git a/clippy_lints/src/utils/mod.rs b/clippy_lints/src/utils/mod.rs
index e83371f8b99a..6d8cfaa5f0d1 100644
--- a/clippy_lints/src/utils/mod.rs
+++ b/clippy_lints/src/utils/mod.rs
@@ -356,8 +356,6 @@ pub fn path_to_res(cx: &LateContext<'_>, path: &[&str]) -> Option<def::Res> {
                 let item_def_id = current_item.res.def_id();
                 if cx.tcx.def_kind(item_def_id) == DefKind::Struct;
                 then {
-                    // Bad `find_map` suggestion. See #4193.
-                    #[allow(clippy::find_map)]
                     return cx.tcx.inherent_impls(item_def_id).iter()
                         .flat_map(|&impl_def_id| cx.tcx.item_children(impl_def_id))
                         .find(|item| item.ident.name.as_str() == *segment)