[PATCH] D101788: [AST] AnyCall: Implement arguments().

[Gábor Horváth via Phabricator via cfe-commits]

xazax.hun added inline comments.


================
Comment at: clang/include/clang/Analysis/AnyCall.h:170
+      return {ME->getArgs(), ME->getNumArgs()};
+    } else if (const auto *CE = dyn_cast<CallExpr>(E)) {
+      return {CE->getArgs(), CE->getNumArgs()};
----------------
As far as I remember `CXXMemberCallExpr` and `CXXOperatorCallExpr` having slightly different behavior with the former not including the implicit object argument in the argument list contrary to the latter. If that is the case, maybe this API should provide some uniformity, e.g. skipping the implicit object argument for `CXXOperatorCallExpr` (and potentially providing a separate function for getting the this/self). What do you think?


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101788/new/

https://reviews.llvm.org/D101788