[PATCH] D58231: [LICM] Support of widenable condition guards in LICM

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 20 08:54:19 PST 2019


reames added a comment.

Does the existing code successfully hoist the widenable_condition call itself out of the loop?  If not, splitting out specifically that part of the transform would be well called for and necessary for unswitch to make progress.  If it does, then the patch as written includes redundant handling which should be removed.


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

https://reviews.llvm.org/D58231





More information about the llvm-commits mailing list