[PATCH] D100684: [X86][CostModel] X86TTIImpl::getMemoryOpCost(): rewrite vector handling again

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 28 06:49:07 PDT 2021


lebedev.ri updated this revision to Diff 341179.
lebedev.ri marked 3 inline comments as done.
lebedev.ri added a comment.

@ABataev thank you for taking a look!
Rebasing, addressing review notes.

As the newly-changed tests show, `getInterleavedMemoryOpCost*()` aren't actually using this function,
which means that before i can look into adding more tuples there,
it would be best to finish with this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100684

Files:
  llvm/lib/Target/X86/X86TargetTransformInfo.cpp
  llvm/test/Analysis/CostModel/X86/interleaved-load-i16-stride-3.ll
  llvm/test/Analysis/CostModel/X86/interleaved-store-i16-stride-3.ll
  llvm/test/Analysis/CostModel/X86/load_store.ll



More information about the llvm-commits mailing list