[PATCH] D10206: Handle resolvable branches in complete loop unroll heuristic.

[Michael Zolotukhin]

mzolotukhin added a comment.

Hi Chandler,

I updated the patch - does it look good now?

I decided not to handle operands in (Base+Offset) form in visitBinaryOperator for now, because I think we could get into troubles even if we only look at SUB (I'm concerned about possible overflow errors here). I think it's possible to do, but we can do it later if we ever need it.

Tests are also added to the patch now (mostly they are from http://reviews.llvm.org/D10208).

Thanks,
Michael


http://reviews.llvm.org/D10206