https://github.com/PiotrZSL approved this pull request.

Overall change is fine.
There is only one usage of isOperatorOverloading, so you could consider 
changing this into:
isOverloadAssigmentOperator and put all those operator kinds in a matcher 
instead of messing with SmallVector.

https://github.com/llvm/llvm-project/pull/84922
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to