[PATCH] D104322: [SCEV] PtrToInt on non-integral pointers is allowed

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

Roughly, we can either end up with SCEV effectively being inoperable for non-integral pointers,
or the users of non-integral pointers having issues with lowering the `ptrtoint`'s produced by SCEV.
I don't know what is worse, and i'm presently not sure i care, but i do believe that we can not
leave the current SCEV casual-ness of treating pointers as integers as-is.
I actually had some of the patches @efriedma posted locally, and i agree with them.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104322