[PATCH] D75145: [PassManager] adjust VectorCombine placement

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 6 03:51:19 PST 2020


bjope added a comment.

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)?

(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





More information about the llvm-commits mailing list