[PATCH] D89456: [SCEV] Introduce SCEVPtrToIntExpr (PR46786)

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 15 12:17:06 PDT 2020


lebedev.ri marked 4 inline comments as done.
lebedev.ri added inline comments.


================
Comment at: polly/test/Isl/CodeGen/ptrtoint_as_parameter.ll:3
+
+; FIXME: what is this trying to check?
+; XFAIL: *
----------------
efriedma wrote:
> I think it's just checking it doesn't crash.
> 
> More generally, I would guess your patch is interfering with polly's code to look through ptrtoint instructions.  If you want, I can fix up the polly stuff after this patch is finished.
> If you want, I can fix up the polly stuff after this patch is finished.

Yes, that would be awesome, thank you! I'm not at all familiar with polly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89456



More information about the llvm-commits mailing list