HazardyKnusperkeks accepted this revision. HazardyKnusperkeks added a comment. This revision is now accepted and ready to land.
In D131978#3727084 <https://reviews.llvm.org/D131978#3727084>, @rymiel wrote: > Added negation unary operator check to TokenAnnotatorTest > > Is this what you meant? Although, I didn't touch the TokenType since it was > already correct before i changed anything It's late (where I am). I thought we had something like `ClosesRequiresClause` for concepts too. But on the other hand, this should affect requires clauses, right? So a test for that would fail before your patch. 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
