[PATCH] D67233: InstCombine: Fix crash on icmp of gep with addrspacecasted null

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineCompares.cpp:908
       !NullPointerIsDefined(I.getFunction(),
                             RHS->getType()->getPointerAddressSpace())) {
     // For most address spaces, an allocation can't be placed at null, but null
----------------
Why doesn't the `NullPointerIsDefined` check catch this and prevent the transformation in the first place?



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

https://reviews.llvm.org/D67233