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

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 21 15:10:00 PDT 2017


hfinkel added inline comments.


================
Comment at: include/llvm/Analysis/TargetTransformInfoImpl.h:793
+      if (!static_cast<T *>(this)->isLoweredToCall(F))
+        return 1;
       return 40;
----------------
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)?


https://reviews.llvm.org/D38104





More information about the llvm-commits mailing list