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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 10 11:23:10 PST 2021


RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86TargetTransformInfo.cpp:3654
+  std::pair<InstructionCost, MVT> ReplicatedVecTyLegalization =
+      TLI->getTypeLegalizationCost(DL, ReplicatedVecTy);
+
----------------
We're not using the cost at all. just the legalized type?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113350



More information about the llvm-commits mailing list