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
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
