[clang] [analyzer] Disable graph-trim-interval by default (PR #111843)

2024-11-21 Thread Donát Nagy via cfe-commits
NagyDonat wrote: It turns out that this commit increases the runtime by ~3% on a set of 6-8 open source projects that we used as a benchmark. (The testing was done on our local fork by @gamesh411, he can provide more accurate data if needed.) Based on this, we decided to temporarily disable th

[clang] [analyzer] Disable graph-trim-interval by default (PR #111843)

2024-11-21 Thread Donát Nagy via cfe-commits
https://github.com/NagyDonat closed https://github.com/llvm/llvm-project/pull/111843 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Disable graph-trim-interval by default (PR #111843)

2024-10-10 Thread Balazs Benics via cfe-commits
steakhal wrote: > > Is it a possible way forward dropping that assert? > > The function that performs the assertion is not part of the static analyzer, > it's a generic graph algorithm from an LLVM support library and the assertion > seems to be a really obvious sanity check. I don't think tha

[clang] [analyzer] Disable graph-trim-interval by default (PR #111843)

2024-10-10 Thread Donát Nagy via cfe-commits
NagyDonat wrote: > Is it a possible way forward dropping that assert? The function that performs the assertion is not part of the static analyzer, it's a generic graph algorithm from an LLVM support library and the assertion seems to be a really obvious sanity check. I don't think that it's re

[clang] [analyzer] Disable graph-trim-interval by default (PR #111843)

2024-10-10 Thread Balazs Benics via cfe-commits
steakhal wrote: And the message I wanted to share xD Is it a possible way forward dropping that assert? https://github.com/llvm/llvm-project/pull/111843 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/lis

[clang] [analyzer] Disable graph-trim-interval by default (PR #111843)

2024-10-10 Thread Balazs Benics via cfe-commits
steakhal wrote: > If someone happens to be heavily affected by this performance loss (and > doesn't fear the crashes) they can re-enable ExplodedNode reclamation by > passing -analyzer-option graph-trim-interval=1000 (the old default) to the > analyzer. Correct me if I'm wrong, but this is an