[clang-tools-extra] [clang-tidy][NFC] Enable 'readability-simplify-boolean-expr' check (PR #158706)

[via cfe-commits]

================
@@ -23,10 +23,10 @@ class ExpandModularHeadersPPCallbacks::FileRecorder {
   /// Records that a given file entry is needed for replaying callbacks.
   void addNecessaryFile(FileEntryRef File) {
     // Don't record modulemap files because it breaks same file detection.
-    if (!(File.getName().ends_with("module.modulemap") ||
-          File.getName().ends_with("module.private.modulemap") ||
-          File.getName().ends_with("module.map") ||
-          File.getName().ends_with("module_private.map")))
+    if (!File.getName().ends_with("module.modulemap") &&
----------------
EugeneZelenko wrote:

I think De Morgan form should be kept, since number of operators is less. Same in other similar places.

https://github.com/llvm/llvm-project/pull/158706