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

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 4 11:36:07 PDT 2016

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.


More information about the llvm-commits mailing list