[PATCH] D23418: [analyzer] Added a reusable constraint system to the CloneDetector

Raphael Isemann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 6 02:47:18 PDT 2017


teemperor added inline comments.


================
Comment at: lib/Analysis/CloneDetection.cpp:395
+
+  for (const Stmt *Child : S->children()) {
+    if (Child == nullptr) {
----------------
I couldn't reproduce, but from what I assume form the warning and the crash that we confused the compiler here by naming both the loop-variable and the parameter 'S'. I renamed it to 'Child' which /should/ fix it.


https://reviews.llvm.org/D23418





More information about the cfe-commits mailing list