[clang] [analyzer][NFC] Merge `BlockInvocationContext` class into `StackFrameContext` (PR #194857)
Balázs Benics via cfe-commits
cfe-commits at lists.llvm.org
Tue May 5 02:17:24 PDT 2026
steakhal wrote:
> `<offtopic>`
>
> > > (By the way, for some reason this separation was not applied among the tests, so the static analyzer tests remained under `test/Analysis` and eventually "crowded out" all the other tests. I moved the last non-analyzer tests from `test/Analysis` to other directories ~ a year ago.)
> >
> >
> > I think a split of `test/Analysis` is slowly brewing. It's not urgent or a blocker though.
>
> There is nothing to **split** in the directory `test/Analysis`: currently it _only_ contains tests for the static analyzer (and perhaps analyzer-dependent utilities like the exploded graph rewriter). The tests of the `lib/Analysis` area are scattered elsewhere (IIRC in `test/Sema` and similar directories).
>
> However, it would be reasonable to **rename** `test/Analysis` to `test/StaticAnalyzer` to avoid confusion.
>
> @steakhal WDYT? `</offtopic>`
Probably yes, but we need to look at the RUN lines to confirm that no clang-frontend diags sneaked in, for example for anything implemented by clang/lib/Sema/AnalysisBasedWarnings.cpp.
Also note that `Scalable` subdirectories would need to be moved out then - for the better.
https://github.com/llvm/llvm-project/pull/194857
More information about the cfe-commits
mailing list