[PATCH] D65182: [analyzer] Add fix-it hint support.
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 3 14:35:34 PDT 2019
NoQ marked an inline comment as done.
NoQ added inline comments.
================
Comment at: clang/test/Analysis/dead-stores.c:11
long idx=abc+3*5; // expected-warning {{never read}} expected-warning{{unused variable 'idx'}}
+ // expected-remark-re at -1{{{{.*}}:11 - {{.*}}:18 - ''}}
}
----------------
gribozavr wrote:
> NoQ wrote:
> > gribozavr wrote:
> > > If the tests ignore the line number anyway, why even print it?
> > I anyway hope that this facility is temporary(c).
> What's the eventual replacement?
I hope we eventually learn how to apply fixits automatically, and then FileCheck fixed files.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65182/new/
https://reviews.llvm.org/D65182
More information about the cfe-commits
mailing list