[all-commits] [llvm/llvm-project] c3f8dd: [NFC][analyzer] Use %clang_analyze_cc1 consistentl...

[Donát Nagy via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c3f8dd1228224841e2fae52cf4f4f4070f2edfac
      https://github.com/llvm/llvm-project/commit/c3f8dd1228224841e2fae52cf4f4f4070f2edfac
  Author: Donát Nagy <donat.nagy at ericsson.com>
  Date:   2025-06-30 (Mon, 30 Jun 2025)

  Changed paths:
    M clang/test/Analysis/ArrayDelete.cpp
    M clang/test/Analysis/DeleteWithNonVirtualDtor.cpp
    M clang/test/Analysis/PR37855.c
    M clang/test/Analysis/aggrinit-cfg-output.cpp
    M clang/test/Analysis/compound-literals.c
    M clang/test/Analysis/ctu-cxxdefaultinitexpr.cpp
    M clang/test/Analysis/ctu-implicit.c
    M clang/test/Analysis/ctu-import-threshold.c
    M clang/test/Analysis/ctu-lookup-name-with-space.cpp
    M clang/test/Analysis/ctu-main.c
    M clang/test/Analysis/ctu-on-demand-parsing.c
    M clang/test/Analysis/ctu-test-import-failure.cpp
    M clang/test/Analysis/exploded-graph-rewriter/dynamic_types.cpp
    M clang/test/Analysis/exploded-graph-rewriter/escapes.c
    M clang/test/Analysis/exploded-graph-rewriter/initializers_under_construction.cpp
    M clang/test/Analysis/exploded-graph-rewriter/l_name_starts_with_l.cpp
    M clang/test/Analysis/exploded-graph-rewriter/macros.c
    M clang/test/Analysis/exploded-graph-rewriter/objects_under_construction.cpp
    M clang/test/Analysis/exploded-graph-rewriter/win_path_forbidden_chars.cpp
    M clang/test/Analysis/gmalloc.c
    M clang/test/Analysis/handle_constructors_for_default_arguments.cpp
    M clang/test/Analysis/handle_constructors_with_new_array.cpp
    M clang/test/Analysis/initialization.c
    M clang/test/Analysis/initialization.cpp
    M clang/test/Analysis/lifetime-cfg-output.cpp
    M clang/test/Analysis/loopexit-cfg-output.cpp
    M clang/test/Analysis/method-arg-decay.m
    M clang/test/Analysis/null-deref-static.m
    M clang/test/Analysis/objc-encode.m
    M clang/test/Analysis/openmp-unsupported.c
    M clang/test/Analysis/pointer-escape-on-conservative-calls.c
    M clang/test/Analysis/scopes-cfg-output.cpp
    M clang/test/Analysis/taint-diagnostic-visitor.c
    M clang/test/Analysis/temporaries-callback-order.cpp
    M clang/test/Analysis/undef-call.c
    M clang/test/Analysis/z3-crosscheck-max-attempts.cpp
    M clang/test/Analysis/z3-crosscheck.c
    M clang/test/Analysis/z3/D83660.c
    M clang/test/Analysis/z3/pretty-dump.c

  Log Message:
  -----------
  [NFC][analyzer] Use %clang_analyze_cc1 consistently (#145895)

A large majority of the LIT tests of the clang static analyzer use RUN
lines with the placeholder `%clang_analyze_cc1` which expands to
`%clang_cc1 -analyze -setup-static-analyzer` where the only effect of
`-setup-static-analyzer` is that it ensures that the macro
`__clang_analyzer__` is defined. However, there were some tests that
used `%clang_cc1 -analyze` directly; this commit changes those to using
`%clang_analyze_cc1` for the sake of consistency.

Previously `%clang_analyze_cc1` did not work within the directory
`exploded-graph-rewriter` (because that directory has its own custom
`lit.local.cfg`) but this problem was eliminated by the recent commit
40cc4379cda6e0d6efe72c55d1968f9cf427a16a, so it was possible to resolve
and delete the FIXME comments asking for this change.

There are a few tests that use `%clang --analyze` or other command-line
flags (e.g. help flags), those are not affected by this change.

This cleanup was discussed in the discourse thread
https://discourse.llvm.org/t/taking-ownership-of-clang-test-analysis/84689/11



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications