[PATCH] D117994: [Clang] Implement multidimentional subscript operator

Corentin Jabot via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 25 00:31:59 PST 2022


cor3ntin added inline comments.


================
Comment at: clang/lib/Sema/SemaAccess.cpp:1799
+    R = SourceRange(ArgExprs.front()->getBeginLoc(),
+                    ArgExprs.front()->getEndLoc());
+  }
----------------
erichkeane wrote:
> So should the 'end' be `ArgExprs.back()->getEndLoc()`?
It should, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117994



More information about the cfe-commits mailing list