[PATCH] D20873: [TTI] Refine default cost for interleaved load groups with gaps

silviu.baranga@arm.com via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 8 06:10:59 PDT 2016


sbaranga added a comment.

This seems reasonable to me. Do you have performance data for this change?

Cheers,
Silviu


================
Comment at: include/llvm/CodeGen/BasicTTIImpl.h:552
@@ +551,3 @@
+    // Return the ceiling of dividing A by B.
+    auto ceil = [](unsigned A, unsigned B) { return A / B + (A % B != 0); };
+
----------------
Would it better to use (A + B - 1) / B?


http://reviews.llvm.org/D20873





More information about the llvm-commits mailing list