[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