[PATCH] D124251: [SimpleLoopUnswitch] Run LICM for nested unswitching tests.

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea added a comment.

As a side note, the fact the freeze instruction needs licm and simple loop unswitch interleaved, will likely affect the plan we had around splitting off the non-trivial unswitch functionality into a separate function pass (the context for that is regarding using divergence analysis: https://reviews.llvm.org/D109762)
It seems like tracking the added freeze instructions and including the functionality of hoisting them as part of the "new" non-trivial SLU pass will resolve this, but I'd like to hear folks feedback on this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124251