[PATCH] D45086: [analyzer] Unroll the loop when it has a unsigned counter.

2018-03-31 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC328919: [analyzer] Unroll the loop when it has a unsigned counter. (authored by henrywong, committed by ). Repository: rC Clang https://reviews.llvm.org/D45086 Files: lib/StaticAnalyzer/Core/LoopUnr

[PATCH] D45086: [analyzer] Unroll the loop when it has a unsigned counter.

2018-03-30 Thread Henry Wong via Phabricator via cfe-commits
MTC updated this revision to Diff 140524. MTC added a comment. Fix typo, `unsinged` -> `unsigned` Repository: rC Clang https://reviews.llvm.org/D45086 Files: lib/StaticAnalyzer/Core/LoopUnrolling.cpp test/Analysis/loop-unrolling.cpp Index: test/Analysis/loop-unrolling.cpp =

[PATCH] D45086: [analyzer] Unroll the loop when it has a unsigned counter.

2018-03-30 Thread Peter Szecsi via Phabricator via cfe-commits
szepet accepted this revision. szepet added a comment. Yepp, thanks for the patch! One small typo below. Comment at: test/Analysis/loop-unrolling.cpp:50 + +int simple_unroll4_unsinged() { + int a[9]; typo: unsigned Repository: rC Clang https://reviews.llv

[PATCH] D45086: [analyzer] Unroll the loop when it has a unsigned counter.

2018-03-30 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov accepted this revision. george.karpenkov added a comment. This revision is now accepted and ready to land. Looks reasonable Repository: rC Clang https://reviews.llvm.org/D45086 ___ cfe-commits mailing list cfe-commits@lists.llvm.

[PATCH] D45086: [analyzer] Unroll the loop when it has a unsigned counter.

2018-03-30 Thread Henry Wong via Phabricator via cfe-commits
MTC created this revision. MTC added reviewers: szepet, a.sidorin, NoQ. Herald added subscribers: cfe-commits, rnkovacs, xazax.hun. Herald added a reviewer: george.karpenkov. MTC edited the summary of this revision. The original implementation in the `LoopUnrolling.cpp` didn't consider the case w