[PATCH] D113319: [clang-format] Improve require and concept handling

[Mark de Wever via Phabricator via cfe-commits]

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