[PATCH] D56303: [clang-tidy] Recognize labelled statements when simplifying boolean exprs

Richard via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 25 20:00:14 PST 2022


LegalizeAdulthood added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.cpp:72-73
 
-const Expr *getBoolLiteral(const MatchFinder::MatchResult &Result,
-                           StringRef Id) {
+static const Expr *getBoolLiteral(const MatchFinder::MatchResult &Result,
+                                  StringRef Id) {
   if (const Expr *Literal = Result.Nodes.getNodeAs<CXXBoolLiteralExpr>(Id))
----------------
FYI

These tabs here (and elsewhere) are phantoms somehow
introduced into the diff by Phabricator.  My local diff file
that I uploaded has zero tabs in it anywhere.



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

https://reviews.llvm.org/D56303



More information about the cfe-commits mailing list