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

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 16 08:29:58 PDT 2021


lebedev.ri added a comment.

In D104322#2822083 <https://reviews.llvm.org/D104322#2822083>, @nikic wrote:

> As a drive-by note, it would be great if you could expand LangRef on non-integral pointers a bit. It made sense to me when it specified that you can't use `ptrtoint` on a non-integral pointer, but without that limitation, it's not really clear to me what the actual difference between a non-integral pointer and a normal one is. What transforms are you not allowed to perform on a non-integral pointer that you can perform on a normal one?

Yes, i would also like to see such documentation, especially if non-integral pointers
are going to be used as an "arbitrary" roadblock for SCEV changes. I would have posted
that in the review for the commit mentioned, but there was none, which also highlights
the problem around non-integral pointer status in llvm :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104322



More information about the llvm-commits mailing list