[PATCH] D55397: [CostModel][X86] Fix overcounting arithmetic cost in illegal types getArithmeticReductionCost

[Craig Topper via Phabricator via llvm-commits]

craig.topper updated this revision to Diff 177090.
craig.topper added a comment.

Do the same thing to the min/max reduction cost.


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

https://reviews.llvm.org/D55397

Files:
  include/llvm/CodeGen/BasicTTIImpl.h
  test/Analysis/CostModel/X86/reduce-add-widen.ll
  test/Analysis/CostModel/X86/reduce-add.ll
  test/Analysis/CostModel/X86/reduce-and-widen.ll
  test/Analysis/CostModel/X86/reduce-and.ll
  test/Analysis/CostModel/X86/reduce-mul-widen.ll
  test/Analysis/CostModel/X86/reduce-mul.ll
  test/Analysis/CostModel/X86/reduce-or-widen.ll
  test/Analysis/CostModel/X86/reduce-or.ll
  test/Analysis/CostModel/X86/reduce-smax-widen.ll
  test/Analysis/CostModel/X86/reduce-smax.ll
  test/Analysis/CostModel/X86/reduce-smin-widen.ll
  test/Analysis/CostModel/X86/reduce-smin.ll
  test/Analysis/CostModel/X86/reduce-umax-widen.ll
  test/Analysis/CostModel/X86/reduce-umax.ll
  test/Analysis/CostModel/X86/reduce-umin-widen.ll
  test/Analysis/CostModel/X86/reduce-umin.ll
  test/Analysis/CostModel/X86/reduce-xor-widen.ll
  test/Analysis/CostModel/X86/reduce-xor.ll
  test/Analysis/CostModel/X86/reduction.ll
  test/Transforms/SLPVectorizer/X86/horizontal-minmax.ll
  test/Transforms/SLPVectorizer/X86/reduction_unrolled.ll