[PATCH] D16343: [BasicAA] Fix for missing must alias information

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 25 13:26:04 PST 2016


reames added a subscriber: reames.
reames requested changes to this revision.
reames added a reviewer: reames.
reames added a comment.
This revision now requires changes to proceed.

This looks like it should be two separate patches.  The mustalias logic should be expressible on 64 bit pointers independent of the wrap around.  The 32 bit wrap around logic I'm a bit unclear on.  I think that it's correct per the lang ref.  At minimum, should be using uint32_t types.  Preferably, you'd rephrase this in a generic way which worked for other pointer sizes (i.e. 48 bit?).


http://reviews.llvm.org/D16343





More information about the llvm-commits mailing list