[PATCH] D67176: [SCEV] Generalize SCEVParameterRewriter to accept SCEV expression as target.

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 11:26:13 PDT 2019


reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM.

I don't really have the context to evaluate this fully, so I'm mostly deferring to author.  I don't see any obvious problems, and the generalization does look useful.  I just don't know the callsite well enough to be certain this has the desired effect, and thus the need to defer to author.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67176





More information about the llvm-commits mailing list