[PATCH] D97961: [Cost]Caonicalize the cost for logical or/and reductions.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 4 08:57:05 PST 2021


ABataev added inline comments.


================
Comment at: llvm/test/Analysis/CostModel/SystemZ/reduce-and.ll:13
+; CHECK-NEXT:  Cost Model: Found an estimated cost of 130 for instruction: %V64 = call i1 @llvm.vector.reduce.and.v64i1(<64 x i1> undef)
+; CHECK-NEXT:  Cost Model: Found an estimated cost of 258 for instruction: %V128 = call i1 @llvm.vector.reduce.and.v128i1(<128 x i1> undef)
 ; CHECK-NEXT:  Cost Model: Found an estimated cost of 1 for instruction: ret i32 undef
----------------
The cost model for SystemZ is not complete/correct, needs to be fixed, that's why there is a regression for it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97961



More information about the llvm-commits mailing list