[PATCH] D26803: Fix known zero bits for addrspacecast

[Matt Arsenault via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Analysis/ValueTracking.cpp:1028-1029
   case Instruction::IntToPtr:
   case Instruction::AddrSpaceCast: // Pointers could be different sizes.
     // Fall through and handle them the same as zext/trunc.
     LLVM_FALLTHROUGH;
----------------
yaxunl wrote:
> arsenm wrote:
> > AddrSpaceCast should probably just be removed from the switch
> Some optimizations will be lost if it is removed.
Those optimizations are also broken


https://reviews.llvm.org/D26803