[PATCH] D37832: Eliminate PHI (int typed) which is only used by inttoptr

Nuno Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 10:30:20 PDT 2017


nlopes added a comment.

In https://reviews.llvm.org/D37832#873621, @nlopes wrote:

> This transformation looks correct to me and goes in the right direction (remove unneeded int2ptr/ptr2int, since they block many optimizations).  The patch can be made a bit more general later.
>  Before the patch goes in, please add a few negative tests; you only have positive ones. Thanks!


Actually I'm less sure now if it's correct. Please give me 1 more day to review this properly.


https://reviews.llvm.org/D37832





More information about the llvm-commits mailing list