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

[Sanjay Patel via Phabricator via cfe-commits]

spatel added a comment.

In https://reviews.llvm.org/D23418#719139, @NoQ wrote:

> Hmm, reverted because i'm seeing crashes on some buildbots (works for me though).
>
> It's crashing somewhere in `saveHash`, seems that some `Stmt`s are null. For instance, http://lab.llvm.org:8011/builders/clang-cmake-aarch64-42vma/builds/5998 :


Not sure if it helps, but there was this build warning:
/home/buildslave/buildslave/clang-cmake-aarch64-39vma/llvm/tools/clang/lib/Analysis/CloneDetection.cpp:395:36: warning: ‘S’ is used uninitialized in this function [-Wuninitialized]


Repository:
  rL LLVM

https://reviews.llvm.org/D23418