Szelethus added a comment.
Herald added a subscriber: ASDenysPetrov.

In D71524#1924508 <https://reviews.llvm.org/D71524#1924508>, @steakhal wrote:

> 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>>"))`


Yup, we could make improvements on `CallDescription` as well :)

> I'm afraid it needs to be a different patch to replace with 
> `CallDescriptionMap`, even though I agree with you.

I don't mean to deploy the map in this patch, but if this lands as-is, the 
proposed logic will have to be revisited again.


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

Reply via email to