[PATCH] D113350: [X86][Costmodel] `getReplicationShuffleCost()`: implement cost model for 32/64 bit-wide elements with AVX512F

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/lib/Target/X86/X86TargetTransformInfo.cpp:3741-3747
+InstructionCost
+X86TTIImpl::getReplicationShuffleCost(Type *EltTy, int ReplicationFactor,
+                                      int VF, ArrayRef<int> Mask,
+                                      TTI::TargetCostKind CostKind) {
+  return BaseT::getReplicationShuffleCost(EltTy, ReplicationFactor, VF, Mask,
+                                          CostKind);
+}
----------------
Yes, we should not need to override this, but then it does not compile,
it seems like that ^ override hides base implementation of this method.
So far my attempts at fixing this have not been successful.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113350