curdeius added a comment.

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);
  }


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

Reply via email to