[PATCH] D26277: [SLP] Fixed cost model for horizontal reduction.

Michael Kuperstein via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 17 10:39:55 PST 2016


mkuper added a comment.

So, now, none of the costs on the tests actually change?
Does that mean that the changes in costs in the previous versions came from the +1?

Can you add a test that demonstrates the cost change? (Preferably in a way that shows what happened - e.g. commit a test with the "bad" cost, and then have a diff with the good one).


https://reviews.llvm.org/D26277





More information about the llvm-commits mailing list