hokein accepted this revision.
hokein added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: clang-tidy/readability/SimplifyBooleanExprCheck.cpp:567
replaceCompoundReturnWithCondition(Result, Compound, true);
- else if (const auto TU = Result.Nodes.getNodeAs<Decl>("top"))
- Visitor(this, Result).TraverseDecl(const_cast<Decl*>(TU));
+ else // MatchOnce matcher
+ Visitor(this, Result).TraverseAST(*Result.Context);
----------------
Ah, what I mean is `assert(MatchedOnce)`.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D54579
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
