rymiel added a comment.

It seems that even with this patch, there is seemingly weird formatting with 
the negation in requires clauses, such as:

  template <typename T>
    requires !F<T>
             int bar(T t);

This is because the "fake parens" of the unary expression opened is never 
closed by the special case clean-up made for requires clauses
However, it turns out this isn't even valid syntax, as it requires parentheses 
for disambiguation. Adding those brings back correct indentation

  template <typename T>
    requires(!F<T>)
  int bar(T t);

Is it okay to ignore this case, as it wouldn't even compile?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131978

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

Reply via email to