[PATCH] D29213: InferAddressSpaces: Handle icmp

[Justin Lebar via Phabricator via llvm-commits]

jlebar added a comment.

Well this one is interesting.

I think I see what you're getting at here; if we can ASC away all uses of a generic pointer except the icmp, then keeping around the generic pointers in icmp is just unnecessary register pressure.

But at least on NVPTX, I think this is only safe if we can transform *both* pointers in the icmp, and moreover only if both pointers are transformed to the same AS.


https://reviews.llvm.org/D29213