[PATCH] D75145: [PassManager] adjust VectorCombine placement

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D75145#1909324 <https://reviews.llvm.org/D75145#1909324>, @bjope wrote:

> When you say "phase ordering bug" that is about not generating as optimized code as expected, right? Not ending up with miscompiles (or compiler crashes)?


That is my understanding, yes, no miscompiles.

> (just curious since we got regressions downstream after this patch... haven't looked deeper at that and it could just be some limitations in our backend, but could be nice to know if it is "safe" for us to revert this downstream while investigating)




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75145