[PATCH] D11718: [TTI] Fix default costs for interleaved accesses

silviu.baranga at arm.com silviu.baranga at arm.com
Mon Aug 3 07:05:46 PDT 2015


sbaranga added inline comments.

================
Comment at: include/llvm/CodeGen/BasicTTIImpl.h:537
@@ -536,2 +536,3 @@
     // Firstly, the cost of load/store operation.
-    unsigned Cost = getMemoryOpCost(Opcode, VecTy, Alignment, AddressSpace);
+    unsigned Cost = static_cast<T *>(this)->getMemoryOpCost(
+        Opcode, VecTy, Alignment, AddressSpace);
----------------
rengolin wrote:
> nitpick: can't you cache the T * to avoid re-using the ugly static_cast?
Perhaps, but using the static_casts seems to be the code style here.


http://reviews.llvm.org/D11718







More information about the llvm-commits mailing list