[PATCH] D73729: [analyzer] AnalyzerOptions: Remove 'fixits-as-remarks'

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 25 03:31:24 PST 2020


NoQ added inline comments.


================
Comment at: clang/include/clang/StaticAnalyzer/Core/AnalyzerOptions.def:310
-ANALYZER_OPTION(bool, ShouldEmitFixItHintsAsRemarks, "fixits-as-remarks",
-                "Emit fix-it hints as remarks for testing purposes",
-                false)
----------------
>>! In D73729#1891015, @xazax.hun wrote:
> Maybe Kristof has some opinion whether we still need this :) He might be up to date whether CodeChecker is using this feature.

Well, it clearly said "for testing purposes" :P

It should be much easier for CodeChecker to read fixits from plists.


Repository:
  rC Clang

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

https://reviews.llvm.org/D73729





More information about the cfe-commits mailing list