[PATCH] D102990: [X86][Costmodel] getMaskedMemoryOpCost(): don't scalarize non-power-of-two vectors with legal element type

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri updated this revision to Diff 347436.
lebedev.ri added a comment.

Fix an obvious bug that we don't actually count the cost of mask extension for some non-power-of-two's.

In D102990#2776878 <https://reviews.llvm.org/D102990#2776878>, @ABataev wrote:

> LG if there are no other opinions

Thank you for the review.
I believe this to be a rather straight-forward change, so i'm going to land this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102990

Files:
  llvm/lib/Target/X86/X86TargetTransformInfo.cpp
  llvm/test/Analysis/CostModel/X86/masked-intrinsic-cost.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D102990.347436.patch
Type: text/x-patch
Size: 89670 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210524/bb92c541/attachment-0001.bin>