[PATCH] D65182: [analyzer] WIP: Add fix-it hint support.

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 24 13:52:18 PDT 2019


NoQ updated this revision to Diff 211595.
NoQ added a comment.

Unforget a `FileCheck`-based test file for the virtual call checker that i already had.

In D65182#1598614 <https://reviews.llvm.org/D65182#1598614>, @Szelethus wrote:

> How does an `-analyzer-config fixits-as-warnings` option sound like for more readable tests?


Yeah, that makes a lot of sense! I guess i'll do fixits-as-remarks instead :D


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65182/new/

https://reviews.llvm.org/D65182

Files:
  clang/include/clang/StaticAnalyzer/Core/BugReporter/BugReporter.h
  clang/include/clang/StaticAnalyzer/Core/BugReporter/PathDiagnostic.h
  clang/lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp
  clang/lib/StaticAnalyzer/Checkers/VirtualCallChecker.cpp
  clang/lib/StaticAnalyzer/Core/BugReporter.cpp
  clang/lib/StaticAnalyzer/Core/PathDiagnostic.cpp
  clang/lib/StaticAnalyzer/Core/PlistDiagnostics.cpp
  clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp
  clang/test/Analysis/Inputs/expected-plists/edges-new.mm.plist
  clang/test/Analysis/Inputs/expected-plists/objc-arc.m.plist
  clang/test/Analysis/Inputs/expected-plists/plist-output.m.plist
  clang/test/Analysis/virtualcall-fixits.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D65182.211595.patch
Type: text/x-patch
Size: 18833 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190724/4c7c86b2/attachment-0001.bin>


More information about the cfe-commits mailing list