[PATCH] D79162: [Analysis] TTI: Add CastContextHint for getCastInstrCost

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:710
       // extending load exists on target.
-      if (I && isa<LoadInst>(I->getOperand(0))) {
+      if (I && CCH == TTI::CastContextHint::Normal) {
         EVT ExtVT = EVT::getEVT(Dst);
----------------
Actually looking at the code again, this needn't check I? Because we are not using it's value directly, and we know the operand it a load now.

That would imply that the `if (!I)` condition above can change to just guard the TLI call too.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79162/new/

https://reviews.llvm.org/D79162