Added a comment to ArraySubscriptExpr to note that the expressions like

"A[4]" are equivalent to "4[A]", and that a test that the expression
returned by "getBase()" has a pointer type is required to resolve which
subexpression is the "true" base expression of the array index.

llvm-svn: 41113

1 files changed, 5 insertions, 0 deletions

diff --git a/clang/include/clang/AST/Expr.h b/clang/include/clang/AST/Expr.h
index 36005223c7e..a5aba76d757 100644
--- a/clang/include/clang/AST/Expr.h
+++ b/clang/include/clang/AST/Expr.h
@@ -375,6 +375,11 @@ public:
     Expr(ArraySubscriptExprClass, t),
     Base(base), Idx(idx), RBracketLoc(rbracketloc) {}
   
+  // NOTE: An array access can be written A[4] or 4[A] (both are equivalent).
+  // In the second case, getBase() actually returns the index and getIdx()
+  // returns the offset.  Only one of the subexpressions will have a pointer
+  // type (the base), so the second case can be identified using the
+  // expression getBase()->getType()->isPointerType().
   Expr *getBase() { return Base; }
   const Expr *getBase() const { return Base; }
   Expr *getIdx() { return Idx; }