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

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

JFYI, I don't object to this change, but I am hesitant about the direction you seem to be indicating.  Please don't use the excuse of our handling of non-integral types being somewhat broken to further break them.  I'll comment on future reviews if appropriate.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104322