[PATCH] D34458: [TTI] Refine the cost of EXT in getUserCost()

[Elena Demikhovsky via Phabricator via llvm-commits]

delena added inline comments.


================
Comment at: llvm/trunk/include/llvm/Analysis/TargetTransformInfoImpl.h:735
         return TTI::TCC_Free;
+      if (isa<SExtInst>(CI) || isa<ZExtInst>(CI) || isa<FPExtInst>(CI))
+        return static_cast<T *>(this)->getExtCost(CI, Operands.back());
----------------
This change affects not only on ARM targets. You changed cost calculation on X86 and we measure significant performance degradation after this patch. I assume that the cost of Ext instruction should be provided by target.


Repository:
  rL LLVM

https://reviews.llvm.org/D34458