HazardyKnusperkeks added a comment.
In D113319#3339801 <https://reviews.llvm.org/D113319#3339801>, @curdeius wrote:
> Yep, confirmed, but I don't know how to fix it.
> Just removing `TT_BinaryOperator` from `resetTokenMetadata` provokes these
> failures:
>
> [ FAILED ] FormatTest.Concepts
> [ FAILED ] FormatTest.RequiresClauses
> [ FAILED ] TokenAnnotatorTest.UnderstandsRequiresClausesAndConcepts
>
> Hopefully you have some idea how to fix it...
>
> Oh, BTW, here's the test for the regression that Krasimir found:
>
> TEST_F(TokenAnnotatorTest, UnderstandsLoops) {
> auto Tokens = annotate("while (p < q && *p == 0) {}");
> EXPECT_EQ(Tokens.size(), 14u) << Tokens;
> EXPECT_TOKEN(Tokens[6], tok::star, TT_UnaryOperator);
> }
This is addressed in D120511 <https://reviews.llvm.org/D120511>.
Repository:
rG LLVM Github Monorepo
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
