[PATCH] D16623: Factor out UnrollAnalyzer to Analysis, and add unit tests for it.

[Michael Zolotukhin via llvm-commits]

mzolotukhin marked 2 inline comments as done.
mzolotukhin added a comment.

Hi Sanjoy,

I updated the patch, please take a look. As for not exposing it as `InstVisitor`: my intention to move this functionality out is just to enable unit-testing for it. And unit-testing in this case is much easier, as we can actually pin-point interesting cases (e.g. given just two instructions `A` and `B`, if we know that `A` is simplified to `C`, do we simplify `B`?). However, I'm happy to reconsider this if there are arguments against it.

Thanks,
Michael


http://reviews.llvm.org/D16623