[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