[PATCH] D78447: [mlir][SCCP] Add support for propagating constants across inter-region control flow

Uday Bondhugula via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 20 12:27:42 PDT 2020


bondhugula accepted this revision.
bondhugula added inline comments.


================
Comment at: mlir/test/Transforms/sccp-structured.mlir:132
+  return %result : i32
+}
----------------
rriddle wrote:
> rriddle wrote:
> > bondhugula wrote:
> > > bondhugula wrote:
> > > > Every loop.if that you have in the test cases has an else. Add one or more without an else block?
> > > Just realized that no values would be yielded without the else. And you are testing for empty regions at one place. 
> > Do you have a suggestion on what you have in mind? An IfOp can only produce results if it has both a then and an else region. Given that IfOp is implicit capture, nothing would really be testing the structure of the IfOp that isn't tested elsewhere.
> Seems we crossed comments. Should no longer be necessary if we restrict that successors are non-empty.
Yes, nothing else to do on this for now. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78447





More information about the llvm-commits mailing list