[PATCH] D82310: Add `BoolLiteralExpression` to SyntaxTree

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 22 09:40:01 PDT 2020


gribozavr2 added inline comments.


================
Comment at: clang/unittests/Tooling/Syntax/TreeTest.cpp:1240
+  true;
+  false;
+}
----------------
C99 has bool literals, but the program should include stdbool.h.

I feel like it is better to make the predicate something like "hasBoolType()" and change the test to include stdbool.h.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82310/new/

https://reviews.llvm.org/D82310





More information about the cfe-commits mailing list