royjacobson planned changes to this revision.
royjacobson added a comment.

I think I didn't get the overload resolution changes quite right, so this will 
probably change a bit.



================
Comment at: clang/lib/Frontend/InitPreprocessor.cpp:698
     Builder.defineMacro("__cpp_multidimensional_subscript", "202110L");
+    Builder.defineMacro("__cpp_static_call_operator", "202207L");
   }
----------------
aaron.ballman wrote:
> Because we're exposing the feature as a language extension in older language 
> modes, I think we probably want to define the macro in older language modes 
> as well, right?
Because supporting static lambdas requires backporting the overload resolution 
change as well, I decided not to support it as an extension, and for 
consistency no regular `static operator()` as well.

I've asked now on the libcxx Discord and they seem to think that it would still 
be useful to provide the regular `static operator()` as an extension, 
especially for C++20 ranges.

I think backporting the overload resolution change should be fine, but it seems 
I haven't implemented it quite right so I'm looking into it.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133659

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to