[PATCH] D138387: [Clang] Implement static operator[]

[Corentin Jabot via Phabricator via cfe-commits]

cor3ntin accepted this revision.
cor3ntin added a comment.
This revision is now accepted and ready to land.

Beside the formatting nitpick this looks good



================
Comment at: clang/test/CodeGenCXX/cxx2b-static-subscript-operator.cpp:14
+  f.operator[](201, 202);
+  Functor{}[301, 302];
+}
----------------
I think this is missing a test like
`Functor::operator[](42, 42)` for completeness


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138387