shuaiwang added a comment. In https://reviews.llvm.org/D50447#1194967, @JonasToth wrote:
> Do you think it is a bad idea? If the variable is not used it is ok to > ignore it in this particular circumstance. Other warnings/check should > deal with such a situation IMHO. > > Am 10.08.2018 um 10:29 schrieb Roman Lebedev via Phabricator: > > > lebedev.ri added a comment. > > > > It seems this ended up silently being a catch-all, with no option to > > control this behavior, and i don't see any comments discussing this.. > > > > Repository: > > > > rL LLVM > > > > https://reviews.llvm.org/D50447 Not sure whether hokein have done that already but I did run through our code base and AFAICT there's no false negative. Repository: rL LLVM https://reviews.llvm.org/D50447 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
