[PATCH] D75145: [PassManager] adjust VectorCombine placement

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 29 08:11:48 PST 2020


fhahn added a comment.

The changes seem relatively save, but I am wondering if the vector combine pass makes the CSE problem more acute? Otherwise it might be better to add the extra EarlyCSE run separately (I'm not sure the name will be quite accurate after the change, it runs quite late now :))


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

https://reviews.llvm.org/D75145





More information about the llvm-commits mailing list