[PATCH] D36962: [StaticAnalyzer] LoopUnrolling: Excluding loops which splits the state (make more branches)
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 28 01:08:44 PDT 2017
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
LGTM!
I guess later it'd be great to comment on every `numTimesReached()` why is it supposed to be reached exactly that many times, as it's often unclear, especially with tricky control flow. Because if somebody accidentally breaks the test, they'd have no idea what it tests or why it should work this way.
https://reviews.llvm.org/D36962
More information about the cfe-commits
mailing list