[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