[PATCH] D38104: [TargetTransformInfo] Handle intrinsic call in getInstructionLatency()

[Guozhi Wei via Phabricator via llvm-commits]

Carrot added inline comments.


================
Comment at: include/llvm/Analysis/TargetTransformInfoImpl.h:793
+      if (!static_cast<T *>(this)->isLoweredToCall(F))
+        return 1;
       return 40;
----------------
hfinkel wrote:
> Actually, do we want to just return 1 here, or do we want to fall through to the logic below (which returns 3 for FP types)?
Good catch! It will give more precise result.


https://reviews.llvm.org/D38104