[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