[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