[PATCH] D75145: [PassManager] adjust VectorCombine placement

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 6 11:02:07 PST 2020


spatel added a comment.

In D75145#1910117 <https://reviews.llvm.org/D75145#1910117>, @lebedev.ri wrote:

> @dmgreen i expect that regression to have been fixed by rG1badf7c33a5d01900c77646f750e2ea11ad8bf5a <https://reviews.llvm.org/rG1badf7c33a5d01900c77646f750e2ea11ad8bf5a>, please confirm.
>  Please do post more findings, if there are any.


Oops...you're moving faster than me or my email inbox, so disregard my earlier question. :)
I'd still like to understand if we have the ideal ARM output now. And maybe we can add a test to PhaseOrdering based on that ARM example, so we have coverage for the larger case.
Also, we may still have other problems as noted by @bjope .


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