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

[Raphael Isemann via Phabricator via cfe-commits]

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