NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land.
This looks negligible compared to compilation time. I wonder if we're now scalable enough to try finding whole-translation-unit clones. ================ Comment at: lib/Analysis/CloneDetection.cpp:402 auto CS = dyn_cast<CompoundStmt>(S); SmallVector<size_t, 8> ChildHashes; ---------------- Hmm, is 8 enough here? People often write large functions. Maybe bump to 32 or 64? https://reviews.llvm.org/D34364 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
