[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