brunodf added a comment.
In D111639#3081504 <https://reviews.llvm.org/D111639#3081504>, @brunodf wrote: > With regard to OO_Subscript, OO_Arrow and OO_Amp, I have tried a number of > things, but I have not succeeded in triggering that an CXXOperatorCallExpr is > created for these operators (they end up as an ArraySubscriptExpr, > CXXDependentScopeMemberExpr and UnaryOperator respectively). At the moment, I > don't know how to test the code paths for these operators in > RebuildCXXOperatorCallExpr. After my submission, I still looked further into cases with these operators, but a CXXOperatorCallExpr is only used for these operators in case an overloaded operator has been determined (so not in case of a dependent type). Also, these operators are not allowed as non-members functions. While a property member could be of a type that has such an overloaded member operator, this would be dubious since it would work on a reference to the temporary returned by the getter of the property member. So while it would be possible to add tests for CXXOperatorCallExpr involving invocation of these operators on property members, I think this would be artificial and not really contribute to the patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111639/new/ https://reviews.llvm.org/D111639 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
