steakhal added a comment. In D71524#1917251 <https://reviews.llvm.org/D71524#1917251>, @Szelethus wrote:
> Are we sure this is what we want? If this is a heuristic, we should document > it well, and even then I'm not sure whether we want it. I'm also pretty sure > this would make the eventual change to `CallDescriptionMap` more difficult, > because the way taintedness is propagated around `std::basic_istream` not > really the property of the global `>>` operator and what its parameters are, > but rather the property of `std::basic_istream<CharT,Traits>::operator>>`, > right? What do you think? I think `CallDescription` can only identify objects/functions which has `IdefntifyerInfo` in them. AFAIK operators don't have such. Though somehow AST matchers of Clang Tidy were triggered with this: `functionDecl(hasName("operator>>"))` I'm afraid it needs to be a different patch to replace with `CallDescriptionMap`, even though I agree with you. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71524/new/ https://reviews.llvm.org/D71524 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits