[PATCH] D24451: [LoopUnroller] Replace UnrollingPreferences::Force with ForceMaxCount + SystemZ getUnrollingPreferences().

Ulrich Weigand via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 27 06:44:14 PDT 2016


uweigand accepted this revision.
uweigand added a comment.
This revision is now accepted and ready to land.

The SystemZ part looks good to me.

Evgeny, thanks for your comments and review of the unroll part!


================
Comment at: lib/Target/SystemZ/SystemZTargetTransformInfo.cpp:243
@@ +242,3 @@
+                                             TTI::UnrollingPreferences &UP) {
+  // Find out if L contains a call, what the machine instruction count
+  // estimate is, and how many stores there are.
----------------
Comment now seems to be out of date, the code no longer computes a machine instruction count.


https://reviews.llvm.org/D24451





More information about the llvm-commits mailing list