[PATCH] D73480: [VectorCombine] new IR transform pass for partial vector ops

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 30 13:30:47 PST 2020


efriedma added a comment.

I think the general idea of a pass that uses vector cost models to do peephole optimizations on vectors makes sense.  A couple general concerns:

1. If we're running it in the middle of the pipeline, we need to be careful the transforms don't conflict with instcombine.
2. The vectorizer's cost model is optimized for throughput; I'm a little worried we'll run into issues with latency if we depend too much on the cost model for scalar code.


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

https://reviews.llvm.org/D73480





More information about the llvm-commits mailing list