[PATCH] D77870: [MLIR] Introduce utility to hoist affine if/else conditions

Uday Bondhugula via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 10 18:51:37 PDT 2020


bondhugula added a comment.

In D77870#1975507 <https://reviews.llvm.org/D77870#1975507>, @bondhugula wrote:

> In D77870#1975018 <https://reviews.llvm.org/D77870#1975018>, @dcaballe wrote:
>
> > Thanks for adding this optimization!
> >  One general comment: this is called loop unswitching in LLVM: https://llvm.org/docs/Passes.html#loop-unswitch-unswitch-loops
> >  It would be great if we could use that terminology for consistency.
>
>
>


Done. The test pass is now called loop unswitching (-test-affine-loop-unswitch) although the utility is still called hoistAffineIf since it's really meant for on an 'if' op.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77870





More information about the llvm-commits mailing list