[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

[Donát Nagy via cfe-commits]

================
@@ -256,6 +257,12 @@ std::vector<SymbolRef> taint::getTaintedSymbolsImpl(ProgramStateRef State,
   if (!Sym)
     return TaintedSymbols;
 
+  // HACK:https://discourse.llvm.org/t/rfc-make-istainted-and-complex-symbols-friends/79570
+  if (const auto &Opts = State->getAnalysisManager().getAnalyzerOptions();
+      Sym->computeComplexity() >= Opts.MaxTaintedSymbolComplexity) {
----------------
NagyDonat wrote:

```suggestion
      Sym->computeComplexity() > Opts.MaxTaintedSymbolComplexity) {
```
I know that this is pedantic bikeshedding, but if the name and documentation claims that "this is the maximal possible complexity of a tainted symbol", then make it actually possible to have a tainted symbol with that complexity.

(Feel free to change the default value to 9 to compensate for this change. Alternatively you could keep the `>=` and rename the analyzer option to discard-taint-at-complexity or something similar.)

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