MyDeveloperDay requested changes to this revision. MyDeveloperDay added inline comments. This revision now requires changes to proceed.
================ Comment at: clang/lib/Format/TokenAnnotator.cpp:2146-2147 + // know how they can be followed by a star or amp. + // co_await, delete - It doesn't make sense to have them followed by a unary + // `+` or `-`. + if (PrevToken->isOneOf(TT_ConditionalExpr, tok::l_paren, tok::comma, ---------------- HazardyKnusperkeks wrote: > Especially here, why should a `+` after `delete` be a binary operator? > How much sense it makes doesn't matter, it is valid c++: > https://gcc.godbolt.org/z/c1x1nn3Ej I'm also trying to understand did you mean you couldn't have case -1: case +1: case +0: case -0: https://gcc.godbolt.org/z/qvE44d5xz Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121754/new/ https://reviews.llvm.org/D121754 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
