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
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
=
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
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.
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