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

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 25 03:40:12 PST 2020


xazax.hun 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)
----------------
NoQ wrote:
> >>! 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.
Fair point :) LGTM!


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