[PATCH] D148732: [SLP][NFC] Test showing a cost estimation issue caused by f82eb7e066f322a231627383fc80522d98ce6181

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 19 12:52:10 PDT 2023


ABataev added inline comments.


================
Comment at: llvm/test/Transforms/SLPVectorizer/X86/buildvector-float-and-extract-lane1.ll:4
+
+; Building a vector of a float (lane 0) and an extract from lane 1 (lane 1)
+; should have a cost of +1 because we need an extra instruction to combine
----------------
vporpo wrote:
> ABataev wrote:
> > Add a cost analysis to the test.
> What do you mean?
Check llvm/test/Transforms/SLPVectorizer/X86/extractelement-multiple-uses.ll as an example


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148732



More information about the llvm-commits mailing list