Skip to content

Commit 8679752

Browse files
committed
[clang-format] Treat f<N | M>(a) as template function call
Fixes #123144.
1 parent 8e31050 commit 8679752

File tree

2 files changed

+10
-5
lines changed

2 files changed

+10
-5
lines changed

clang/lib/Format/TokenAnnotator.cpp

+4-4
Original file line numberDiff line numberDiff line change
@@ -252,10 +252,10 @@ class AnnotatingParser {
252252
// parameters.
253253
// FIXME: This is getting out of hand, write a decent parser.
254254
if (MaybeAngles && InExpr && !Line.startsWith(tok::kw_template) &&
255-
Prev.is(TT_BinaryOperator)) {
256-
const auto Precedence = Prev.getPrecedence();
257-
if (Precedence > prec::Conditional && Precedence < prec::Relational)
258-
MaybeAngles = false;
255+
Prev.is(TT_BinaryOperator) &&
256+
(Prev.isOneOf(tok::pipepipe, tok::ampamp) ||
257+
Prev.getPrecedence() == prec::Equality)) {
258+
MaybeAngles = false;
259259
}
260260
if (Prev.isOneOf(tok::question, tok::colon) && !Style.isProto())
261261
SeenTernaryOperator = true;

clang/unittests/Format/TokenAnnotatorTest.cpp

+6-1
Original file line numberDiff line numberDiff line change
@@ -685,7 +685,7 @@ TEST_F(TokenAnnotatorTest, UnderstandsNonTemplateAngleBrackets) {
685685
EXPECT_TOKEN(Tokens[4], tok::less, TT_BinaryOperator);
686686
EXPECT_TOKEN(Tokens[8], tok::greater, TT_BinaryOperator);
687687

688-
Tokens = annotate("return A < B ^ A > B;");
688+
Tokens = annotate("return A < B != A > B;");
689689
ASSERT_EQ(Tokens.size(), 10u) << Tokens;
690690
EXPECT_TOKEN(Tokens[2], tok::less, TT_BinaryOperator);
691691
EXPECT_TOKEN(Tokens[6], tok::greater, TT_BinaryOperator);
@@ -3671,6 +3671,11 @@ TEST_F(TokenAnnotatorTest, TemplateInstantiation) {
36713671
EXPECT_TOKEN(Tokens[2], tok::less, TT_TemplateOpener);
36723672
EXPECT_TOKEN(Tokens[6], tok::greater, TT_TemplateCloser);
36733673

3674+
Tokens = annotate("return FixedInt<N | M>(foo);");
3675+
ASSERT_EQ(Tokens.size(), 12u) << Tokens;
3676+
EXPECT_TOKEN(Tokens[2], tok::less, TT_TemplateOpener);
3677+
EXPECT_TOKEN(Tokens[6], tok::greater, TT_TemplateCloser);
3678+
36743679
Tokens = annotate("return std::conditional_t<T::value == U::value, T, U>{};");
36753680
ASSERT_EQ(Tokens.size(), 21u) << Tokens;
36763681
EXPECT_TOKEN(Tokens[4], tok::less, TT_TemplateOpener);

0 commit comments

Comments
 (0)