[PATCH] D16721: [clang-tidy] Move implicit-cast-in-loop check to upstream.

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 29 07:04:03 PST 2016


alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.

Looks good with one nit. I'll fix and commit the patch.

Thank you!


================
Comment at: clang-tidy/performance/ImplicitCastInLoopCheck.cpp:79
@@ +78,3 @@
+  // code at the end).
+  if (IsNonTrivialImplicitCast(Materialized->getTemporary())) {
+    ReportAndFix(Result.Context, VD, OperatorCall);
----------------
No braces needed around single-line `if` bodies.


Repository:
  rL LLVM

http://reviews.llvm.org/D16721





More information about the cfe-commits mailing list