Mordante added inline comments.

================
Comment at: clang/lib/Format/Format.cpp:1212
   LLVMStyle.ReferenceAlignment = FormatStyle::RAS_Pointer;
+  // This is open for discussions! When will LLVM adapt C++20?
+  LLVMStyle.RequiresClausePositionForClasses = FormatStyle::RCPS_OwnLine;
----------------
HazardyKnusperkeks wrote:
> Mordante wrote:
> > Note the libc++ already uses C++20 code. We don't use clang-format 
> > officially, but I use it for new code. (Mainly the `std::format` related 
> > code.)
> And is there a consensus about formatting requires clauses?
No there's not. But maybe it's interesting to discuss that after the patch 
lands to see how the different options look in real code.


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

https://reviews.llvm.org/D113319

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

Reply via email to