NoQ added a comment. LGTM, thanks! Minor nit included. I hope our matchers actually cover all the cases (not a big deal if they don't though).
================ Comment at: lib/StaticAnalyzer/Core/LoopUnrolling.cpp:26 +#define MAXIMUM_STEP_UNROLLED 128 + ---------------- I'd prefer a (`static`) `const int`, dealing preprocessor doesn't seem to be necessary here. https://reviews.llvm.org/D37181 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
