summaryrefslogtreecommitdiff
path: root/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp
diff options
context:
space:
mode:
authorBalazs Benics <benicsbalazs@gmail.com>2024-08-21 14:24:56 +0200
committerGitHub <noreply@github.com>2024-08-21 14:24:56 +0200
commit848658955a9d2d42ea3e319d191e2dcd5d76c837 (patch)
treef8cc92b62d58501251192d7859811af04ab443c7 /lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp
parentad435bcc14f42dc97286c717cd12446a0facb2ee (diff)
[analyzer] Limit `isTainted()` by skipping complicated symbols (#105493)
As discussed in https://discourse.llvm.org/t/rfc-make-istainted-and-complex-symbols-friends/79570/10 Some `isTainted()` queries can blow up the analysis times, and effectively halt the analysis under specific workloads. We don't really have the time now to do a caching re-implementation of `isTainted()`, so we need to workaround the case. The workaround with the smallest blast radius was to limit what symbols `isTainted()` does the query (by walking the SymExpr). So far, the threshold 10 worked for us, but this value can be overridden using the "max-tainted-symbol-complexity" config value. This new option is "deprecated" from the getgo, as I expect this issue to be fixed within the next few months and I don't want users to override this value anyways. If they do, this message will let them know that they are on their own, and the next release may break them (as we no longer recognize this option if we drop it). Mitigates #89720 CPP-5414
Diffstat (limited to 'lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp')
0 files changed, 0 insertions, 0 deletions