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

Roy Jacobson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 23 10:11:47 PST 2022


royjacobson marked 3 inline comments as done.
royjacobson added inline comments.


================
Comment at: clang/lib/Sema/SemaOverload.cpp:14462-14467
+              RLoc, CurFPFeatureOverrides());
+        else
+          TheCall =
+              CallExpr::Create(Context, FnExpr.get(), MethodArgs, ResultTy, VK,
+                               RLoc, CurFPFeatureOverrides());
+
----------------
cor3ntin wrote:
> This looks weird, did you clang-format?
clang-format insists on this, personally I wouldn't write it like this either :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138387



More information about the cfe-commits mailing list