[PATCH] D32563: Improve code placement algorithm in Reassociate pass.

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 27 15:44:24 PDT 2017


dberlin added a comment.

The whole point of starting to rewrite-reassociate as n-ary reassociate was to have a better framework to handle better reassociation and placement issues.
Reassociate is supposed to be a simple, and dumb, reassociate.

Trying to integrate code placement into it seems like a ... bad plan.
It's neither structured in a way that you can easily make such a thing fast, nor can you really do a good job of it.


https://reviews.llvm.org/D32563





More information about the llvm-commits mailing list