[clang-tidy] Incorrect suggestion for readability-simplify-boolean-expr

Nobody has claimed this yet.

Reproducer: https://godbolt.org/z/cKfnavMvb

```
#include
#include
struct Foo {
std::string appName;
int maxCopy;
};

int main()
{
std::string app;
Foo fileMeta;
int maxCopy;
bool x = !(app == fileMeta.appName && maxCopy == fileMeta.maxCopy);
}
```

clang-tidy shows:

```
:13:10: warning: Value stored to 'x' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
13 | bool x = !(app == fileMeta.appName && maxCopy == fileMeta.maxCopy);
| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
:13:10: note: Value stored to 'x' during its initialization is never read
13 | bool x = !(app == fileMeta.appName && maxCopy == fileMeta.maxCopy);
| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
:13:14: warning: boolean expression can be simplified by DeMorgan's theorem [readability-simplify-boolean-expr]
13 | bool x = !(app == fileMeta.appName && maxCopy == fileMeta.maxCopy);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| ! || !=
2 warnings generated.
```

But it should be `app != fileMeta.appName || maxCopy != fileMeta.maxCopy)`

Start with the clang-tidy readability-simplify-boolean-expr check and reproduce the warning using the linked Compiler Explorer example. Compare the suggested DeMorgan transformation with the expected app != fileMeta.appName || maxCopy != fileMeta.maxCopy result, then add or update coverage for this expression so the diagnostic and replacement are correct.

Written by the indexing model from the issue text.