[PATCH] D15677: Cost for Gather and Scatter operations.

Elena Demikhovsky via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 26 23:35:55 PST 2015


delena marked 11 inline comments as done.

================
Comment at: ../lib/Target/X86/X86TargetTransformInfo.cpp:1412
@@ +1411,3 @@
+         "Masked Gather / Scatter should be legal on this target");
+  Type *PtrTy = isa<VectorType>(Ptr->getType()) ? cast<VectorType>(Ptr->getType())->getElementType() :
+    Ptr->getType();
----------------
Ayal wrote:
> Is 'cast' needed? See pattern used below in getGatherScatterOpCost, or use dyn_cast instead?
I removed this function


Repository:
  rL LLVM

http://reviews.llvm.org/D15677





More information about the llvm-commits mailing list