[PATCH] D124677: [ConstantFold] Don't convert getelementptr to ptrtoint+inttoptr

Nuno Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 29 09:10:44 PDT 2022


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

Sounds great, thanks! We should avoid introducing ptr2int at all costs.

Note for other reviewers: the new code is covered by existing tests. It's there to avoid regressing on cases where no int2ptr is introduced on new pointers, but we can still get rid of the gep. I think keeping this case is find as it doesn't escape more than the original code.


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

https://reviews.llvm.org/D124677



More information about the llvm-commits mailing list