[all-commits] [llvm/llvm-project] f69c74: [analyzer] FixItHint: Apply and test hints with th...

Charusso via All-commits all-commits at lists.llvm.org
Tue Mar 3 21:27:45 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: f69c74db34f42c20c167b8fb0f93ec05a0e77b45
      https://github.com/llvm/llvm-project/commit/f69c74db34f42c20c167b8fb0f93ec05a0e77b45
  Author: Charusso <dabis.csaba98 at gmail.com>
  Date:   2020-03-04 (Wed, 04 Mar 2020)

  Changed paths:
    M clang/include/clang/StaticAnalyzer/Core/AnalyzerOptions.def
    M clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp
    M clang/lib/StaticAnalyzer/Frontend/CMakeLists.txt
    M clang/test/Analysis/analyzer-config.c
    A clang/test/Analysis/check-analyzer-fixit.py
    A clang/test/Analysis/virtualcall-fixit.cpp
    M clang/test/lit.cfg.py

  Log Message:
  -----------
  [analyzer] FixItHint: Apply and test hints with the Clang-Tidy's script

Summary:
This patch introduces a way to apply the fix-its by the Analyzer:
`-analyzer-config apply-fixits=true`.

The fix-its should be testable, therefore I have copied the well-tested
`check_clang_tidy.py` script. The idea is that the Analyzer's workflow
is different so it would be very difficult to use only one script for
both Tidy and the Analyzer, the script would diverge a lot.
Example test: `// RUN: %check-analyzer-fixit %s %t -analyzer-checker=core`

When the copy-paste happened the original authors were:
@alexfh, @zinovy.nis, @JonasToth, @hokein, @gribozavr, @lebedev.ri

Reviewed By: NoQ, alexfh, zinovy.nis

Differential Revision: https://reviews.llvm.org/D69746




More information about the All-commits mailing list