[PATCH] D21928: Aliasing of constant pointers (inttoptr Const) for BasicAA

[Eli Friedman via llvm-commits]

eli.friedman added a subscriber: eli.friedman.
eli.friedman added a comment.

This is missing a check for the address-space; address-spaces other than zero can behave in strange ways.

This doesn't correctly account for the width of the pointer: it's possible two pointers could overlap via wraparound.

I don't this is really a good idea, anyway: this seems likely to break someone's code which accidentally forgot to mark a pointer volatile, and unlikely to actually help performance.


http://reviews.llvm.org/D21928