[PATCH] D35888: Changed basic cost of Store operation on X86.

[Elena Demikhovsky via Phabricator via llvm-commits]

delena added a comment.

> What about having getLoadCost? I think there will be a reasonable question why there is getStoreCost but no getLoadCost. 
>  Or can we have the overloaded version of TargetTransformInfo::getMemoryOpCost:
> 
> int TargetTransformInfo::getMemoryOpCost(const Instruction *I)

I don't think that we want to extend interface without any special need. Load is 1 Uop on X86, and, probably, on other targets, so everyone is happy with the common getOperatopnCost().


Repository:
  rL LLVM

https://reviews.llvm.org/D35888