[PATCH] D92488: [LICM] Add a maximum for the number of instructions to be hoisted

[Philip Reames via Phabricator via llvm-commits]

reames requested changes to this revision.
reames added a comment.
This revision now requires changes to proceed.

Hard reject.  This patch will not be accepted.  This is a major design change.

Hoisting operations out of loops is treated as a canonicalization transform in LLVM.  If the register pressure is problematic, the MachineLICM pass is supposed to undo the transformation.  If you have cases where that is not happening, please explore why.

If after doing so, you still feel this is the right approach, please raise the design change on llvm-dev.


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

https://reviews.llvm.org/D92488