[PATCH] D13402: Refactor loop unrolling pass and add optimization remarks

[hfinkel@anl.gov via llvm-commits]

hfinkel added a comment.

In http://reviews.llvm.org/D13402#269178, @mzolotukhin wrote:

> Hi Mark,


...

> One more question: loop-unrolling is currently executed twice in our O3 pipeline, and in theory it's possible that we'll unroll the same loop twice (e.g. partially the first time and completely the second time). In this case we'll emit two remarks for the same loop, which might be confusing for the user. Any thought on how we can avoid doing this?


No, this should not be possible. We only do full unrolling early in the pipeline. All forms of unrolling are enabled later in the pipeline.


http://reviews.llvm.org/D13402